Skip to content

Conversation

@boomanaiden154
Copy link
Contributor

This reverts commit 18f7e03.

Reapplies #160133.

This was reverted originally because it was causing compilation crashes in ControlHeightReduction. It was reverted again because fix in CntrolHeightReduction was causing more problems. Everything should be in working order now, so relanding.

This reverts commit 18f7e03.

Reapplies llvm#160133.

This was reverted originally because it was causing compilation crashes
in ControlHeightReduction. It was reverted again because fix in
CntrolHeightReduction was causing more problems. Everything should be in
working order now, so relanding.
@boomanaiden154 boomanaiden154 added the skip-precommit-approval PR for CI feedback, not intended for review label Sep 25, 2025
@llvmbot llvmbot added the clang Clang issues not falling into any other category label Sep 25, 2025
@llvmbot
Copy link
Member

llvmbot commented Sep 25, 2025

@llvm/pr-subscribers-clang

Author: Aiden Grossman (boomanaiden154)

Changes

This reverts commit 18f7e03.

Reapplies #160133.

This was reverted originally because it was causing compilation crashes in ControlHeightReduction. It was reverted again because fix in CntrolHeightReduction was causing more problems. Everything should be in working order now, so relanding.


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

1 Files Affected:

  • (modified) clang/cmake/caches/PGO.cmake (+1-1)
diff --git a/clang/cmake/caches/PGO.cmake b/clang/cmake/caches/PGO.cmake
index 15bc755d110d1..d6471160037c1 100644
--- a/clang/cmake/caches/PGO.cmake
+++ b/clang/cmake/caches/PGO.cmake
@@ -5,7 +5,7 @@ set(LLVM_ENABLE_PROJECTS "clang;lld" CACHE STRING "")
 set(LLVM_ENABLE_RUNTIMES "compiler-rt;libcxx;libcxxabi;libunwind" CACHE STRING "")
 
 set(LLVM_TARGETS_TO_BUILD Native CACHE STRING "")
-set(BOOTSTRAP_LLVM_BUILD_INSTRUMENTED ON CACHE BOOL "")
+set(BOOTSTRAP_LLVM_BUILD_INSTRUMENTED IR CACHE BOOL "")
 set(CLANG_BOOTSTRAP_TARGETS
   generate-profdata
   stage2

@boomanaiden154 boomanaiden154 merged commit e2cab7c into llvm:main Sep 25, 2025
12 checks passed
@boomanaiden154 boomanaiden154 deleted the reland-irpgo-again branch September 25, 2025 15:56
@llvm-ci
Copy link
Collaborator

llvm-ci commented Sep 25, 2025

LLVM Buildbot has detected a new failure on builder clang-ppc64-aix running on aix-ppc64 while building clang at step 6 "test-build-unified-tree-check-all".

Full details are available at: https://lab.llvm.org/buildbot/#/builders/64/builds/5910

Here is the relevant piece of the build log for the reference
Step 6 (test-build-unified-tree-check-all) failure: test (failure)
******************** TEST 'LLVM-Unit :: Support/./SupportTests/14/204' FAILED ********************
Script(shard):
--
GTEST_OUTPUT=json:/home/llvm/llvm-external-buildbots/workers/aix-ppc64/clang-ppc64-aix/build/unittests/Support/./SupportTests-LLVM-Unit-18416066-14-204.json GTEST_SHUFFLE=0 GTEST_TOTAL_SHARDS=204 GTEST_SHARD_INDEX=14 /home/llvm/llvm-external-buildbots/workers/aix-ppc64/clang-ppc64-aix/build/unittests/Support/./SupportTests
--

Script:
--
/home/llvm/llvm-external-buildbots/workers/aix-ppc64/clang-ppc64-aix/build/unittests/Support/./SupportTests --gtest_filter=ProgramEnvTest.TestExecuteNoWaitTimeoutPolling
--
Note: Google Test filter = ProgramEnvTest.TestExecuteNoWaitTimeoutPolling
[==========] Running 1 test from 1 test suite.
[----------] Global test environment set-up.
[----------] 1 test from ProgramEnvTest
/home/llvm/llvm-external-buildbots/workers/aix-ppc64/clang-ppc64-aix/llvm-project/llvm/unittests/Support/ProgramTest.cpp:414: Failure
Expected: (LoopCount) > (1u), actual: 1 vs 1
LoopCount should be >1


/home/llvm/llvm-external-buildbots/workers/aix-ppc64/clang-ppc64-aix/llvm-project/llvm/unittests/Support/ProgramTest.cpp:414
Expected: (LoopCount) > (1u), actual: 1 vs 1
LoopCount should be >1



********************


mahesh-attarde pushed a commit to mahesh-attarde/llvm-project that referenced this pull request Oct 3, 2025
…llvm#160711)

This reverts commit 18f7e03.

Reapplies llvm#160133.

This was reverted originally because it was causing compilation crashes
in ControlHeightReduction. It was reverted again because fix in
CntrolHeightReduction was causing more problems. Everything should be in
working order now, so relanding.
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 skip-precommit-approval PR for CI feedback, not intended for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants