Skip to content

[SYCL] PR 3 - Remove FPGA attributes from SYCL FE#21735

Open
premanandrao wants to merge 3 commits intointel:syclfrom
premanandrao:remote_fpga_pr3
Open

[SYCL] PR 3 - Remove FPGA attributes from SYCL FE#21735
premanandrao wants to merge 3 commits intointel:syclfrom
premanandrao:remote_fpga_pr3

Conversation

@premanandrao
Copy link
Copy Markdown
Contributor

This removes the following attributes:

[[intel::ivdep]]
[[intel::initiation_interval]]
[[intel::ii]]
[[intel::max_concurrency]]
[[intel::speculated_iterations]]
[[intel::max_reinvocation_delay]]
[[intel::disable_loop_pipelining]]
[[intel::enable_loop_pipelining]]

@premanandrao premanandrao requested a review from a team as a code owner April 10, 2026 19:31
@premanandrao premanandrao marked this pull request as draft April 10, 2026 19:32
This removes the following attributes:

[[intel::ivdep]]
[[intel::initiation_interval]]
[[intel::ii]]
[[intel::max_concurrency]]
[[intel::speculated_iterations]]
[[intel::max_reinvocation_delay]]
[[intel::disable_loop_pipelining]]
[[intel::enable_loop_pipelining]]
@premanandrao premanandrao marked this pull request as ready for review April 10, 2026 20:04
@premanandrao
Copy link
Copy Markdown
Contributor Author

#21102 requested that we split the large set of changes into manageable chunks.
Previous PRs removing some FPGA loop attributes here:
#21710
#21722

let PragmaAttributeSupport = 0;
}

def SYCLIntelIVDep : StmtAttr {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was curious about what ivdep is and google gave me this https://www.intel.com/content/www/us/en/docs/oneapi-fpga-add-on/developer-guide/2025-0/ivdep-attribute.html . I wonder if need to ask someone to remove this from somewhere too

return LHS->getSafelenValue() > RHS->getSafelenValue();
}
}];
let Documentation = [SYCLIntelIVDepAttrDocs];
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait, should we also remove all the docs?

Copy link
Copy Markdown
Contributor

@Fznamznon Fznamznon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requesting changes so we don't miss docs removal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants