[PATCH] D125000: [RS4GC] Cache IsKnownBase for bases and BDVs (NFC)

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 11 05:09:32 PDT 2022


mkazantsev added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp:694
 
+static bool isKnownBase(Value *V, IsKnownBaseMapTy &KnownBases) {
+  return KnownBases[V];
----------------
const IsKnownBase ... ?


================
Comment at: llvm/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp:695
+static bool isKnownBase(Value *V, IsKnownBaseMapTy &KnownBases) {
+  return KnownBases[V];
+}
----------------
Pls add assert with descriptive message what happens if the value is missing


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125000/new/

https://reviews.llvm.org/D125000



More information about the llvm-commits mailing list