mizvekov added inline comments.
Herald added a subscriber: Peiming.
================
Comment at: bolt/tools/CMakeLists.txt:11-13
+macro(add_bolt_tool_symlink name)
+ llvm_add_tool_symlink(BOLT ${ARGV})
+endmacro()
----------------
Instead of each project having to define a different macro for this, why not
just do this based on the current project in scope?
I think this would be a less invasive change.
FWIW my review can only help in these small matters of coding, I can't really
review this from the perspective of the intended effect, I am just a clang
developer and I never have to package or install LLVM I built myself, I just
test stuff from the build dir :)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117977/new/
https://reviews.llvm.org/D117977
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
