Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Re-sync with upstream, fix BitCast noise #7

Merged
merged 7 commits into from
Mar 13, 2025

Conversation

AlexVlx
Copy link
Contributor

@AlexVlx AlexVlx commented Mar 13, 2025

No description provided.

svenvh and others added 7 commits March 10, 2025 09:45
Update after llvm-project commit `979c275097a6 ("[IR] Store Triple in
Module (NFC) (#129868)", 2025-03-06)`.
… -1) (#3036)

Corrected sunormal checking logic in is_fpclass intrinsic issubnormal(V)
==> unsigned(abs(V) - 1) < (all mantissa bits set)
corrected the testfile to check the corrected logic
…(#3050)

Instead of checking that the image operands are equal to the LOD mask,
check that the LOD bit is in the image operands. This way an LOD value
of zero may be ignored even when image operands contains bits other than
the LOD bit, such as for the SPIR-V 1.6 nontemporal image operand.

fixes #3049
llvm-spirv.cpp was relying on transitive includes for these.

Also remove an unneeded `<set>` include.
Update the pattern after llvm-project commit da0f9e75d858 ("Reland: [MC]
output inlined-at debug info (#106230) (#130306)", 2025-03-11).
@AlexVlx AlexVlx requested review from ronlieb and searlmc1 March 13, 2025 21:02
Copy link
Collaborator

@searlmc1 searlmc1 left a comment

Choose a reason for hiding this comment

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

LGTM

@searlmc1 searlmc1 merged commit 07a5e00 into amd-staging Mar 13, 2025
searlmc1 added a commit that referenced this pull request Mar 13, 2025
* Absorb upstream carnage around `Triple`. (#6)

* Re-sync with upstream, fix BitCast noise (#7)

* Update after llvm::Triple changes (#3046)

Update after llvm-project commit `979c275097a6 ("[IR] Store Triple in
Module (NFC) (#129868)", 2025-03-06)`.

* Corrected subnormal checking logic issubnormal(V) ==> unsigned(abs(V) -1) (#3036)

Corrected sunormal checking logic in is_fpclass intrinsic issubnormal(V)
==> unsigned(abs(V) - 1) < (all mantissa bits set)
corrected the testfile to check the corrected logic

* properly handle LOD values when nontemporal image operand is present (#3050)

Instead of checking that the image operands are equal to the LOD mask,
check that the LOD bit is in the image operands. This way an LOD value
of zero may be ignored even when image operands contains bits other than
the LOD bit, such as for the SPIR-V 1.6 nontemporal image operand.

fixes #3049

* [NFC] Add some missing includes (#3045)

llvm-spirv.cpp was relying on transitive includes for these.

Also remove an unneeded `<set>` include.

* Emit error for LLVM bfloat type (#3047)

Signed-off-by: Arvind Sudarsanam <[email protected]>

* Update DebugInfo test after LLVM change (#3060)

Update the pattern after llvm-project commit da0f9e75d858 ("Reland: [MC]
output inlined-at debug info (#106230) (#130306)", 2025-03-11).

---------

Signed-off-by: Arvind Sudarsanam <[email protected]>
Co-authored-by: Sven van Haastregt <[email protected]>
Co-authored-by: sumesh-s-mcw <[email protected]>
Co-authored-by: Ben Ashbaugh <[email protected]>
Co-authored-by: Arvind Sudarsanam <[email protected]>

---------

Signed-off-by: Arvind Sudarsanam <[email protected]>
Co-authored-by: Alex Voicu <[email protected]>
Co-authored-by: Sven van Haastregt <[email protected]>
Co-authored-by: sumesh-s-mcw <[email protected]>
Co-authored-by: Ben Ashbaugh <[email protected]>
Co-authored-by: Arvind Sudarsanam <[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.

6 participants