jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.
LG, as it doesn't affect core features.
================
Comment at: flang/examples/FeatureList/FeatureList.cpp:40-43
+ const auto [it, ins] = frequencies.insert({name, 1});
+ if (!ins) {
+ frequencies[name] = it->second + 1;
+ }
----------------
you could used &it and avoid a second lookup in line 42.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143704/new/
https://reviews.llvm.org/D143704
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
