[PATCH] D71182: [NFC][llvm][MIRVRegNamerUtils] Making some stylistic changes to MIRVRegNamerUtils.cpp
Puyan Lotfi via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 9 10:52:49 PST 2019
plotfi marked 2 inline comments as done.
plotfi added inline comments.
================
Comment at: llvm/lib/CodeGen/MIRVRegNamerUtils.cpp:21
- auto VReg = I->first;
- auto Rename = I->second;
+ for (auto &E : VRM) {
+ unsigned VReg = E.first;
----------------
compnerd wrote:
> Why not const? The map itself isn’t modified.
ditto
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71182/new/
https://reviews.llvm.org/D71182
More information about the llvm-commits
mailing list