[PATCH] D73513: [memtag] Plug in stack safety analysis.

Vitaly Buka via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 3 18:22:27 PST 2020


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


================
Comment at: llvm/lib/Analysis/StackSafetyAnalysis.cpp:670
+      if (AllocaRange.contains(AS.Use.Range)) {
+        AS.AI->setMetadata(M.getMDKindID("stack-safe"),
+                           MDNode::get(M.getContext(), None));
----------------
Could you please add basic test in StackSafetyAnalysis to checks "stack-safe" in IR?



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73513





More information about the cfe-commits mailing list