[PGO] Remove calls to __llvm_orderfile_dump() in instrprof-api.c test

[qiongsiwu]

#78285 added a test which calls __llvm_orderfile_dump(), a functionality that is not supported on many platforms. This PR removes the call to __llvm_orderfile_dump() to avoid it failing on unsupported platforms, and turn on the test for Windows, so we test the rest of the API that are supported.

[llvmbot]

@llvm/pr-subscribers-pgo

Author: Qiongsi Wu (qiongsiwu)

Changes

#78285 added a test which calls __llvm_orderfile_dump(), a functionality that is not supported on many platforms. This PR removes the call to __llvm_orderfile_dump() to avoid it failing on unsupported platforms, and turn on the test for Windows, so we test the rest of the API that are supported.

---

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

1 Files Affected:

• (modified) compiler-rt/test/profile/instrprof-api.c (-7)

diff --git a/compiler-rt/test/profile/instrprof-api.c b/compiler-rt/test/profile/instrprof-api.c
index fedec2d1afc48de..07c5b2f45390793 100644
--- a/compiler-rt/test/profile/instrprof-api.c
+++ b/compiler-rt/test/profile/instrprof-api.c
@@ -1,6 +1,3 @@
-// UNSUPPORTED: target={{.*windows.*}}
-// __llvm_orderfile_dump() is not supported on Windows.
-
 // Testing profile generate.
 // RUN: %clang_profgen %s -S -emit-llvm -o - | FileCheck %s --check-prefix=PROFGEN
 // RUN: %clang_pgogen %s -S -emit-llvm -o - | FileCheck %s --check-prefix=PROFGEN
@@ -38,12 +35,8 @@ int main() {
     return 2;
   // PROFGEN: %{{.*}} = call {{(signext )*}}i32 @__llvm_profile_dump()
   // PROFUSE-NOT: %{{.*}} = call {{(signext )*}}i32 @__llvm_profile_dump()
-  __llvm_orderfile_dump();
-  // PROFGEN: %{{.*}} = call {{(signext )*}}i32 @__llvm_orderfile_dump()
-  // PROFUSE-NOT: %{{.*}} = call {{(signext )*}}i32 @__llvm_orderfile_dump()
   return z + bar() - 11;
 }
 
 // PROFUSE-NOT: declare void @__llvm_profile_set_filename(ptr noundef)
 // PROFUSE-NOT: declare signext i32 @__llvm_profile_dump()
-// PROFUSE-NOT: declare signext i32 @__llvm_orderfile_dump()

[qiongsiwu]

Landing this PR. I will keep an eye on the Windows (https://lab.llvm.org/buildbot/#/builders/127) / Solaris(https://lab.llvm.org/buildbot/#/builders/72) buildbots that have reported problems before.

[qiongsiwu]

Windows build succeeded https://lab.llvm.org/buildbot/#/builders/127/builds/61284.

[qiongsiwu]

Solaris build succeeded https://lab.llvm.org/buildbot/#/builders/72/builds/2033.

[rorth]

Solaris build succeeded https://lab.llvm.org/buildbot/#/builders/72/builds/2033.

Right: that's the sparcv9 one. Solaris/amd64 is fine, too. Thanks.