MaskRay added inline comments.
================
Comment at: llvm/lib/Passes/PassBuilderPipelines.cpp:1797
addPGOInstrPasses(MPM, Level, /* RunProfileGen */ true,
- /* IsCS */ true, PGOOpt->CSProfileGenFile,
- PGOOpt->ProfileRemappingFile,
+ /* IsCS */ true, PGOOpt->AtomicCounterUpdate,
+ PGOOpt->CSProfileGenFile, PGOOpt->ProfileRemappingFile,
----------------
qiongsiwu1 wrote:
> MaskRay wrote:
> > The canonical way to specify the parameter name is `/*IsCS=*/true`
> Ah thanks for pointing it out! Let me fix it in a later NFC patch if that is
> OK. There are quite a few changes required and they would make the actual
> feature harder to see.
I think it's better to fix the argument style while you are modifying it.
Otherwise, there may not be not much value to do a batch change for this file.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157280/new/
https://reviews.llvm.org/D157280
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
