Skip to content

Conversation

@wlei-llvm
Copy link
Contributor

-fprofile-sample-use is not supported on AIX, which caused a CI failure.

@wlei-llvm wlei-llvm requested a review from abhina-sree October 29, 2024 20:30
@llvmbot llvmbot added the clang Clang issues not falling into any other category label Oct 29, 2024
@llvmbot
Copy link
Member

llvmbot commented Oct 29, 2024

@llvm/pr-subscribers-clang

Author: Lei Wang (wlei-llvm)

Changes

-fprofile-sample-use is not supported on AIX, which caused a CI failure.


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

1 Files Affected:

  • (modified) clang/test/CodeGen/pgo-cold-function-coverage.c (+1-1)
diff --git a/clang/test/CodeGen/pgo-cold-function-coverage.c b/clang/test/CodeGen/pgo-cold-function-coverage.c
index fd1e1e7e14cda5..3003cdc3e15e02 100644
--- a/clang/test/CodeGen/pgo-cold-function-coverage.c
+++ b/clang/test/CodeGen/pgo-cold-function-coverage.c
@@ -1,7 +1,7 @@
 // Test -fprofile-generate-cold-function-coverage 
 
 // RUN: rm -rf %t && split-file %s %t
-// RUN: %clang -O2 -fprofile-generate-cold-function-coverage=/xxx/yyy/ -fprofile-sample-accurate -fprofile-sample-use=%t/pgo-cold-func.prof  -S -emit-llvm -o - %t/pgo-cold-func.c | FileCheck %s
+// RUN: %clang --target=x86_64 -O2 -fprofile-generate-cold-function-coverage=/xxx/yyy/ -fprofile-sample-accurate -fprofile-sample-use=%t/pgo-cold-func.prof  -S -emit-llvm -o - %t/pgo-cold-func.c | FileCheck %s
 
 // CHECK: @__llvm_profile_filename = {{.*}} c"/xxx/yyy/default_%m.profraw\00"
 

@wlei-llvm wlei-llvm merged commit cc60c46 into llvm:main Oct 30, 2024
8 checks passed
steelannelida pushed a commit that referenced this pull request Oct 31, 2024
wlei-llvm added a commit to wlei-llvm/llvm-project that referenced this pull request Oct 31, 2024
smallp-o-p pushed a commit to smallp-o-p/llvm-project that referenced this pull request Nov 3, 2024
NoumanAmir657 pushed a commit to NoumanAmir657/llvm-project that referenced this pull request Nov 4, 2024
`-fprofile-sample-use` is not supported on AIX, which caused a CI
failure.
NoumanAmir657 pushed a commit to NoumanAmir657/llvm-project that referenced this pull request Nov 4, 2024
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