[PATCH] D64098: [NFC][clang] Refactor getCompilationPhases step 1: Move list of phases into Types.def table.
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jul 18 11:22:49 PDT 2019
aaron.ballman added inline comments.
Comment at: clang/lib/Driver/Types.cpp:305
+ for (auto Phase : getInfo(Id).Phases)
> aaron.ballman wrote:
> > Can't you use the local `Phases` object instead of calling `getInfo()` again? This seems like it wants to be `P = Phases;`
> Oh yeah sure. I can change that. Anything else pop out at you?
You can drop the `P.clear()` as part of that assignment, but otherwise, no.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits