[PATCH] D118580: [BOLT][CMAKE] Add extra BOLT_INCLUDE_TESTS condition for merge-fdata emit-relocs option

Amir Ayupov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 30 22:03:35 PST 2022


Amir created this revision.
Amir added reviewers: maksfb, rafauler, yota9.
Herald added subscribers: ayermolo, mgorny.
Amir requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Only enable --emit-relocs linker option for merge-fdata target if tests are enabled.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D118580

Files:
  bolt/tools/merge-fdata/CMakeLists.txt


Index: bolt/tools/merge-fdata/CMakeLists.txt
===================================================================
--- bolt/tools/merge-fdata/CMakeLists.txt
+++ bolt/tools/merge-fdata/CMakeLists.txt
@@ -9,6 +9,6 @@
 set_target_properties(merge-fdata PROPERTIES FOLDER "BOLT")
 
 # Emit relocations for BOLT meta test (bolt/test/runtime/meta-merge-fdata.test)
-if (UNIX AND NOT APPLE)
+if (BOLT_INCLUDE_TESTS AND UNIX AND NOT APPLE)
   target_link_options(merge-fdata PRIVATE LINKER:--emit-relocs)
 endif()


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D118580.404424.patch
Type: text/x-patch
Size: 504 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220131/80d5147d/attachment.bin>


More information about the llvm-commits mailing list