manojgupta added a comment. Thanks,
Noticed a few typos. Rest lgtm but deferring to other reviewers for now for approval. ================ Comment at: clang/lib/CodeGen/BackendUtil.cpp:1110 PTO.SLPVectorization = CodeGenOpts.VectorizeSLP; + PTO.CallGrpahProfile = CodeGenOpts.CallGraphProfile; PTO.Coroutines = LangOpts.Coroutines; ---------------- Typo: PTO.CallGrpahProfile -> PTO.CallGraphProfile ================ Comment at: clang/lib/CodeGen/BackendUtil.cpp:1518 Conf.PTO.SLPVectorization = CGOpts.VectorizeSLP; + Conf.PTO.CallGrpahProfile = CGOpts.CallGraphProfile; ---------------- Same typo. ================ Comment at: llvm/include/llvm/Passes/PassBuilder.h:111 + /// true. + bool CallGrpahProfile; }; ---------------- CallGrpahProfile -> CallGraphProfile Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62627/new/ https://reviews.llvm.org/D62627 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
