[PATCH] D80771: [MTE] Convert StackSafety into analysis

[Vitaly Buka via Phabricator via cfe-commits]

vitalybuka marked 2 inline comments as done.
vitalybuka added inline comments.


================
Comment at: clang/test/Driver/memtag_lto.c:126
+  // XUNSAFE: [4]: full-set
+  // XSAFE: [4]: [0,4)
   int x;
----------------
eugenis wrote:
> vitalybuka wrote:
> > eugenis wrote:
> > > Alloca order can easily change in the future. Not sure how to make this better. Perhaps simply remove the numbers and test that the function has one alloca with [0, 4) and one with [0, 1)? This looks specific enough.
> > -DAG?
> Well, that, but also the actual numbers "[1]" and "[4]". Where are "2" and "3", and how likely is 4 to turn into 5 in the future?
[4] is not value number, but alloca size


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80771