Skip to content

[PowerPC] Remove SPE from triple in VSX test #147184

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

guy-david
Copy link
Contributor

This test does not fail because the PPCSubtarget was not initialized, and otherwise a fatal error occurs:
"SPE and traditional floating point cannot both be enabled.".

Tested by adding -mllvm -enable-matrix to the existing command-line.

This test does not fail because the PPCSubtarget was not initialized,
and otherwise a fatal error occurs:
"SPE and traditional floating point cannot both be enabled.".

Tested by adding `-mllvm -enable-matrix` to the existing command-line.
@guy-david guy-david requested a review from MaskRay July 6, 2025 12:23
@llvmbot llvmbot added clang Clang issues not falling into any other category backend:PowerPC labels Jul 6, 2025
@llvmbot
Copy link
Member

llvmbot commented Jul 6, 2025

@llvm/pr-subscribers-backend-powerpc

@llvm/pr-subscribers-clang

Author: Guy David (guy-david)

Changes

This test does not fail because the PPCSubtarget was not initialized, and otherwise a fatal error occurs:
"SPE and traditional floating point cannot both be enabled.".

Tested by adding -mllvm -enable-matrix to the existing command-line.


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

1 Files Affected:

  • (modified) clang/test/CodeGen/PowerPC/builtins-ppc-fpconstrained.c (+1-1)
diff --git a/clang/test/CodeGen/PowerPC/builtins-ppc-fpconstrained.c b/clang/test/CodeGen/PowerPC/builtins-ppc-fpconstrained.c
index 838db02415fe5..b46fa9f2cf157 100644
--- a/clang/test/CodeGen/PowerPC/builtins-ppc-fpconstrained.c
+++ b/clang/test/CodeGen/PowerPC/builtins-ppc-fpconstrained.c
@@ -11,7 +11,7 @@
 // RUN: -S -ffp-exception-behavior=strict \
 // RUN: -o - %s | FileCheck --check-prefix=CHECK-ASM \
 // RUN: --check-prefix=FIXME-CHECK  %s
-// RUN: %clang_cc1 -triple powerpcspe -ffp-exception-behavior=strict \
+// RUN: %clang_cc1 -triple powerpc -ffp-exception-behavior=strict \
 // RUN: -target-feature +vsx -fexperimental-strict-floating-point -emit-llvm \
 // RUN: %s -o - | FileCheck --check-prefix=CHECK-CONSTRAINED %s
 

@guy-david guy-david requested review from chenzheng1030 and stefanp-synopsys and removed request for MaskRay July 6, 2025 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend:PowerPC clang Clang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants