[PATCH] D159153: Support atomic write operations in stack safety

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 31 14:23:15 PDT 2023


vitalybuka accepted this revision.
vitalybuka added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Analysis/StackSafetyAnalysis.cpp:412
+      auto RecordWrite = [&](unsigned ValOp) {
+        if (V == I->getOperand(ValOp)) {
+          // Stored the pointer - conservatively assume it may be unsafe.
----------------
maybe instead of introducing getNewValOperandIndex
just change RecordWrite(Value* operand) ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159153



More information about the llvm-commits mailing list