Skip to content

Conversation

@ronlieb
Copy link
Collaborator

@ronlieb ronlieb commented Nov 11, 2025

No description provided.

naveen-seth and others added 14 commits November 10, 2025 21:24
…w clangOptions library" (llvm#167374)

This relands llvm#167348.

The original PR was reverted due to a reported build failure, which was
later diagnosed as a local issue in the developer’s checkout or build
state. See discussion here:
llvm#163659 (comment)

No additional changes have been made in this reland.
This isn't reachable today but will come into play once we reorder
passes for llvm#147352 and llvm#147351.

Note that the `CBufferRowIntrin` helper struct is copied from the
`DXILCBufferAccess` pass, but it will be removed from there when we
simplify that pass in llvm#147351
trans bf16 insts do not support clamp and omod
…m#167276)

Preparing to moving most of implementation out of the header file.

* llvm#167280

---------

Co-authored-by: Naveen Seth Hanig <naveen.hanig@outlook.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…lvm#167383)

Re-land: Add test for Complex imag literal GNU extension after updating
the name
…166459)

First, for internal variables, they are always global, so use the global
AS by default unless specified otherwise. We can't really use `0` as a
default like we do now because that has an actual meaning on some
targets, so we really need specified vs unspecified, so I used
`std::optional` which is already used in many places in OMPIRBuilder.

Second, for the critical lock variable, add an addrspace cast if needed.

Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>
@ronlieb ronlieb requested review from a team and dpalermo November 11, 2025 03:38
@z1-cciauto
Copy link
Collaborator

@ronlieb ronlieb requested a review from lamb-j as a code owner November 11, 2025 04:49
@z1-cciauto
Copy link
Collaborator

@ronlieb ronlieb closed this Nov 11, 2025
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.