[PATCH] D143704: [flang] Feature list plugin

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 16 14:47:49 PDT 2023


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



More information about the cfe-commits mailing list