RKSimon added inline comments.
================ Comment at: llvm/lib/LTO/LTOBackend.cpp:234 + if (TM) + TM->setPGOOption(PGOOpt); ---------------- @xur Why does TM need a null check - afaict it's already been dereferenced in the call tree, and will be again below. Noticed in a scan-build warning: https://llvm.org/reports/scan-build/report-LTOBackend.cpp-runNewPMPasses-34-35dfe9.html#EndPath Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107878/new/ https://reviews.llvm.org/D107878 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits