tejohnson added inline comments.
================ Comment at: clang/lib/CodeGen/BackendUtil.cpp:995 + // TODO: Consider passing the MemoryProfileOutput to the pass builder via + // the PGOOptions, and set this up there. ---------------- vitalybuka wrote: > if this is registerOptimizerLastEPCallback, it can be next to other > sanitizers, in addSanitizers() It could be, but while it uses the sanitizer common support, it isn't logically a sanitizer. Is there an advantage to setting it up there? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151593/new/ https://reviews.llvm.org/D151593 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
