[PATCH] D67417: [LICM/AST] Check if the AliasAny set is removed from the tracker.

Alina Sbirlea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 11 15:22:32 PDT 2019


asbirlea added inline comments.


================
Comment at: test/Transforms/LICM/pr38513.ll:3
+; REQUIRES: asserts
+
+define void @f1() {
----------------
bjope wrote:
> uabelho wrote:
> > Add a few CHECKs here perhaps?
> > Right now FileCheck complains there are none:
> > ```
> > error: no check strings found with prefix 'CHECK:'
> > ```
> Yes, CHECK:s are needed. And then "REQUIRES: asserts" can be removed.
The test hits an assertion. It won't trigger without asserts.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D67417





More information about the llvm-commits mailing list