[PATCH] D54543: [stack-safety] Inter-Procedural Analysis implementation

JF Bastien via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 26 15:52:06 PST 2018


jfb reopened this revision.
jfb added a comment.
This revision is now accepted and ready to land.

Please fix the build break introduced.



================
Comment at: llvm/trunk/lib/Analysis/StackSafetyAnalysis.cpp:466
+    assert(!CS.Range.isEmptySet() &&
+           "Param range can't be empty-set, invalid access range");
+
----------------
This doesn't build:

```
../lib/Analysis/StackSafetyAnalysis.cpp:465:16: error: no member named 'Range' in '(anonymous namespace)::PassAsArgInfo'
    assert(!CS.Range.isEmptySet() &&
            ~~ ^
```

I commented it out in r347616. Please fix.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D54543





More information about the llvm-commits mailing list