Skip to content

Conversation

advay168
Copy link
Contributor

@advay168 advay168 commented Mar 7, 2025

This PR adds support for compiling builtin variables like threadIdx down to the appropriate intrinsic.

@advay168
Copy link
Contributor Author

advay168 commented Mar 8, 2025

This PR depends on #1444 for the test to compile as warpSize is defined as a constant in the header file.

@advay168 advay168 force-pushed the main branch 2 times, most recently from 048ff30 to 0c1c8ed Compare March 8, 2025 15:54
Copy link
Member

@bcardosolopes bcardosolopes left a comment

Choose a reason for hiding this comment

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

Overall looks good, need fixing naming issues.

Copy link

github-actions bot commented Mar 11, 2025

✅ With the latest revision this PR passed the C/C++ code formatter.

@advay168
Copy link
Contributor Author

advay168 commented Mar 12, 2025

I think it should be good to go now? @bcardosolopes

@bcardosolopes bcardosolopes merged commit 33b9a2d into llvm:main Mar 12, 2025
6 checks passed
lanza pushed a commit that referenced this pull request Mar 18, 2025
This PR adds support for compiling builtin variables like `threadIdx`
down to the appropriate intrinsic.

---------

Co-authored-by: Aidan Wong <[email protected]>
Co-authored-by: anominos <[email protected]>
terapines-osc-cir pushed a commit to Terapines/clangir that referenced this pull request Sep 2, 2025
This PR adds support for compiling builtin variables like `threadIdx`
down to the appropriate intrinsic.

---------

Co-authored-by: Aidan Wong <[email protected]>
Co-authored-by: anominos <[email protected]>
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