Skip to content

Conversation

@z1-cciauto
Copy link
Collaborator

No description provided.

ssahasra and others added 12 commits December 9, 2025 07:36
This is follow-up of comment
llvm#168318 (comment)

libclc OpenCL library is already compiled with flag
`-fdeclare-opencl-builtins -finclude-default-header`.
…ring (llvm#171118)

The lowering for HOM_Epilog did not transfer explicit register defs from
the pseudo-instruction to the generated helper calls. MachineVerifier
would complain if a following tail call uses one of the restored CSRs.
This scenario occurs in code generated by the Swift compiler, where X20
is used to pass swiftself.

This patch fixes the issue by adding the missing defs back to the helper
call as implicit defs.
…69875)

Add following extensions:
cl_intel_bfloat16_conversion
cl_intel_subgroup_buffer_prefetch
cl_intel_subgroup_local_block_io
cl_intel_subgroups_char
cl_intel_subgroups_long

This allows targets to expose these extensions via
getSupportedOpenCLOpts and ensures macros are defined when enabled.
We have disabled unrolling for vectorized loops in llvm#151525 but this
PR only checked the instruction type.

For some loops, there is no instruction with vector type but they
are still vector operations (just like the memset zero test in the
precommit test).

Here we check the operands as well to cover these cases.
…ables. (llvm#171247)

No in-tree target exercises this, but it's needed for CHERI, and I
believe its correctness is verifiable by inspection.

Co-authored-by: Alex Richardson <[email protected]>
The test introduced in llvm#171118 has `llc` inadvertently producing an
output into the same dir as the test file itself. Most build bots don't
clean up the local git repo, which is assumed to not be written by build
+ test, and patch on top (for build performance reasons), which means
the produced output from the aforementioned PR is treated as a test from
here onwards, by all bots. Since it's missing `RUN` lines, we get
errors, for example
 https://lab.llvm.org/buildbot/#/builders/108/builds/20674

This patch fixes the `llc` line and also removes the `.s`. This avoids
all bot maintainers go restart their bots. Then, the cleanup is removed
in llvm#171256.
… conditional (llvm#170752)

We expect the type test to succeed.
@z1-cciauto z1-cciauto requested a review from a team December 9, 2025 05:04
@z1-cciauto
Copy link
Collaborator Author

@ronlieb ronlieb closed this Dec 9, 2025
@ronlieb ronlieb reopened this Dec 9, 2025
@z1-cciauto
Copy link
Collaborator Author

@ronlieb ronlieb merged commit ecd0748 into amd-staging Dec 9, 2025
14 checks passed
@ronlieb ronlieb deleted the upstream_merge_202512090004 branch December 9, 2025 13:28
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.

9 participants