Skip to content

[clang codegen] Fix the ci fail for PR98704 #99267

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

Merged
merged 1 commit into from
Jul 17, 2024
Merged

Conversation

vfdff
Copy link
Contributor

@vfdff vfdff commented Jul 17, 2024

Different targets may have different flag on arguments, so restrict the triple to avoid ci fail.

Different targets may have different flag on arguments,
so restrict the triple to avoid ci fail.
@llvmbot llvmbot added the clang Clang issues not falling into any other category label Jul 17, 2024
@llvmbot
Copy link
Member

llvmbot commented Jul 17, 2024

@llvm/pr-subscribers-clang

Author: Allen (vfdff)

Changes

Different targets may have different flag on arguments, so restrict the triple to avoid ci fail.


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

1 Files Affected:

  • (modified) clang/test/CodeGen/math-libcalls-tbaa.cpp (+2-2)
diff --git a/clang/test/CodeGen/math-libcalls-tbaa.cpp b/clang/test/CodeGen/math-libcalls-tbaa.cpp
index 5b93079492bc5..0b231d474df77 100644
--- a/clang/test/CodeGen/math-libcalls-tbaa.cpp
+++ b/clang/test/CodeGen/math-libcalls-tbaa.cpp
@@ -1,7 +1,7 @@
 // NOTE: Assertions have been autogenerated by utils/update_cc_test_checks.py UTC_ARGS: --version 5
 
-// RUN:  %clang_cc1 -fmath-errno -O3 -emit-llvm -o - -x c++ %s | FileCheck %s -check-prefixes=CHECK,NoNewStructPathTBAA
-// RUN:  %clang_cc1 -fmath-errno -O3 -new-struct-path-tbaa -emit-llvm -o - -x c++ %s | FileCheck %s -check-prefixes=CHECK,NewStructPathTBAA
+// RUN:  %clang_cc1 -triple=aarch64-unknown-linux-gnu -fmath-errno -O3 -emit-llvm -o - -x c++ %s | FileCheck %s -check-prefixes=CHECK,NoNewStructPathTBAA
+// RUN:  %clang_cc1 -triple=aarch64-unknown-linux-gnu -fmath-errno -O3 -new-struct-path-tbaa -emit-llvm -o - -x c++ %s | FileCheck %s -check-prefixes=CHECK,NewStructPathTBAA
 
 extern "C" float expf(float);
 

@vfdff
Copy link
Contributor Author

vfdff commented Jul 17, 2024

try to fix #98704

@vfdff vfdff changed the title [clang codegen] Fix the ci fail for PR96025 [clang codegen] Fix the ci fail for PR98704 Jul 17, 2024
@vfdff vfdff requested a review from efriedma-quic July 17, 2024 03:26
@vfdff vfdff merged commit 484fdb9 into llvm:main Jul 17, 2024
7 of 8 checks passed
@efriedma-quic
Copy link
Collaborator

This seems fine.

yuxuanchen1997 pushed a commit that referenced this pull request Jul 25, 2024
Summary:
Different targets may have different flag on arguments, so restrict the
triple to avoid ci fail.

Test Plan: 

Reviewers: 

Subscribers: 

Tasks: 

Tags: 


Differential Revision: https://phabricator.intern.facebook.com/D60250813
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.

3 participants