vitalybuka 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. ---------------- tejohnson wrote: > 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? no advantages. it has runtime similar to sanitizer, so I rather thing about this as sanitizer :) 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
