Skip to content

[NFC] Ensure the SPIRV target is enabled for test that uses AMDGCNSPIRV #129298

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

Closed
wants to merge 6 commits into from

Conversation

AlexVlx
Copy link
Contributor

@AlexVlx AlexVlx commented Feb 28, 2025

Fixes breakage found when trying to merge #128360.

@llvmbot llvmbot added the clang Clang issues not falling into any other category label Feb 28, 2025
@llvmbot
Copy link
Member

llvmbot commented Feb 28, 2025

@llvm/pr-subscribers-clang

Author: Alex Voicu (AlexVlx)

Changes

Fixes breakage found when trying to merge #128360.


Full diff: https://github.com/llvm/llvm-project/pull/129298.diff

1 Files Affected:

  • (modified) clang/test/Headers/__clang_hip_math.hip (+1)
diff --git a/clang/test/Headers/__clang_hip_math.hip b/clang/test/Headers/__clang_hip_math.hip
index a375ea47b530d..d448ab134ca4d 100644
--- a/clang/test/Headers/__clang_hip_math.hip
+++ b/clang/test/Headers/__clang_hip_math.hip
@@ -1,5 +1,6 @@
 // NOTE: Assertions have been autogenerated by utils/update_cc_test_checks.py
 // REQUIRES: amdgpu-registered-target
+// REQUIRES: spirv-registered-target
 
 // Test without OCML_BASIC_ROUNDED_OPERATIONS
 // RUN: %clang_cc1 -include __clang_hip_runtime_wrapper.h \

@AlexVlx AlexVlx closed this Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang Clang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants