Skip to content

Commit d66d506

Browse files
committed
Revert "Use -fprofile-use= when TEST_SUITE_USE_IR_PGO is enabled"
This reverts commit c1ea19f. It looks like this is passing the wrong paths to -fprofile-instr-use now. Previously it passed ${target_path}.profdata. The new path (${TEST_SUITE_PROFILE_PATH}/${target}.profdat) doesn't point to the correct location when building via LNT in some configurations. Reverting for now to get bots back to green.
1 parent c1ea19f commit d66d506

File tree

1 file changed

+2
-7
lines changed

1 file changed

+2
-7
lines changed

cmake/modules/TestSuite.cmake

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -54,13 +54,8 @@ function(llvm_test_executable_no_test target)
5454
append_target_flags(LINK_LIBRARIES ${target} ${LDFLAGS})
5555
set(target_path ${CMAKE_CURRENT_BINARY_DIR}/${target})
5656
if(TEST_SUITE_PROFILE_USE)
57-
if(TEST_SUITE_USE_IR_PGO)
58-
append_target_flags(COMPILE_FLAGS ${target} -fprofile-use=${TEST_SUITE_PROFILE_PATH}/${target}.profdata)
59-
append_target_flags(LINK_LIBRARIES ${target} -fprofile-use=${TEST_SUITE_PROFILE_PATH}/${target}.profdata)
60-
else()
61-
append_target_flags(COMPILE_FLAGS ${target} -fprofile-instr-use=${TEST_SUITE_PROFILE_PATH}/${target}.profdata)
62-
append_target_flags(LINK_LIBRARIES ${target} -fprofile-instr-use=${TEST_SUITE_PROFILE_PATH}/${target}.profdata)
63-
endif()
57+
append_target_flags(COMPILE_FLAGS ${target} -fprofile-instr-use=${target_path}.profdata)
58+
append_target_flags(LINK_LIBRARIES ${target} -fprofile-instr-use=${target_path}.profdata)
6459
endif()
6560

6661
llvm_codesign(${target})

0 commit comments

Comments
 (0)