[PATCH] D29875: Kaleidoscope-Ch7: Add missing transformutils to llvm-config params

[NAKAMURA Takumi via Phabricator via llvm-commits]

chapuni added a comment.

"x86" (one of "native") may induct TransformUtils, though.

I wonder libraries on llvm-config could describe minimal set, or "Used libraries in toy.cpp"
If the latter,  we should give, to llvm-config, LINK_COMPONENTS in llvm/examples/Kaleidoscope/Chapter7/CMakeLists.txt.

That is, I don't think it'd be appropriate just to add TransformUtils to the command line of llvm-config in the doc.

FYI, I recommend LINK_COMPONENTS should describe "Libraries referred from objects to be linked".
In future, clang-modules would help it. (modules should know libs to link)


https://reviews.llvm.org/D29875