vitalybuka added inline comments.
================ Comment at: clang/lib/CodeGen/BackendUtil.cpp:1070-1071 + // ThinLTOIndexFile is provideds so we must be in ThinLTO PostLink. + // For -O0 ThinLTO PreLink does basic optimization and triggers + // OptimizerLastEPCallbacks. PostLink will not + // run optimizations and this callback should ---------------- aeubanks wrote: > We could fix this. It doesn't make sense for only -O0 to run > OptimizerLastEPCallbacks prelink. Would that help simplify this? Do you mean move optimizer of -O0 into PostLink as well? Yes, this would be more consistent. Still I guess it should a separate patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96456/new/ https://reviews.llvm.org/D96456 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
