stephenneuendorffer added inline comments.
================ Comment at: mlir/examples/standalone/CMakeLists.txt:35 +endif() + include(TableGen) ---------------- mehdi_amini wrote: > mehdi_amini wrote: > > I am a bit unsure that it is desirable that it is desirable to need this > > change? > > This requires every single user of LLVM to do this. Is there a way this can > > be done in the call to `find_package(LLVM)` instead? > (Doc for the usual way to integrate LLVM: > https://llvm.org/docs/CMake.html#embedding-llvm-in-your-project ) I tend to agree with Mehdi. I think stuff like this which is *required* to use LLVM should get pushed into what LLVM exports. (Actually there is a bunch of other configuration stuff which also needs to get pushed into the export, otherwise configuration defaults are all wrong and things like that.) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79219/new/ https://reviews.llvm.org/D79219 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
