Fix issue, as pointed out by Russell: https://reviews.llvm.org/D71786#2270702
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
One small comment. Apart from that, LGTM.
Thanks
llvm/CMakeLists.txt | ||
---|---|---|
516 | I think it might make more sense to put all of the new LLVM_INTEGRATED_CRT_ALLOC code further up now, such as here, so it is not under "Define options to control the inclusion..." section which is (almost entirely) boolean options to enable build targets. |
I think it might make more sense to put all of the new LLVM_INTEGRATED_CRT_ALLOC code further up now, such as here, so it is not under "Define options to control the inclusion..." section which is (almost entirely) boolean options to enable build targets.