[PATCH] D113160: [stack-safety] Check SCEV constraints at memory instructions.

[Kirill Stoimenov via Phabricator via llvm-commits]

kstoimenov added inline comments.


================
Comment at: llvm/lib/Analysis/StackSafetyAnalysis.cpp:875
+        for (const auto &A : KV.second.AccessIsUnsafe)
+          Info->AccessIsUnsafe[A.first] |= A.second;
       }
----------------
If converted to set this loop could be: 

Info->AccessIsUnsafe[A.first].insert(KV.second.AccessIsUnsafe.first, KV.second.AccessIsUnsafe.end);

There is also set::merge in C++ 17, but it will move the elements from the second set I think.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113160