Skip to content

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

Merged
KornevNikita merged 4 commits intointel:syclfrom
premanandrao:remote_fpga_pr3
Apr 15, 2026
Merged

[SYCL] PR 3 - Remove FPGA attributes from SYCL FE#21735
KornevNikita merged 4 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
@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 Author

Choose a reason for hiding this comment

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

Thanks, I removed from AttrDocs.td. I will go back and see if I missed removing any from the previous two PRs; and if it is from the merged one, will create a new PR for that.

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

@KornevNikita
Copy link
Copy Markdown
Contributor

@premanandrao could you please resolve the merge conflicts

@github-actions
Copy link
Copy Markdown
Contributor

@intel/llvm-gatekeepers please consider merging

@KornevNikita KornevNikita merged commit 44a2ed2 into intel:sycl Apr 15, 2026
45 of 46 checks passed
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.

3 participants