[PATCH] [LoopAccesses] Add -analyze support

Adam Nemet anemet at apple.com
Mon Feb 16 17:39:06 PST 2015


================
Comment at: test/Analysis/LoopAccessAnalysis/unsafe-and-rt-checks.ll:2
@@ +1,3 @@
+; RUN: opt -loop-accesses -analyze < %s | FileCheck %s
+; RUN: opt -loop-accesses -analyze -debug-only=loop-accesses < %s 2>&1 | FileCheck %s --check-prefix=DEBUG
+
----------------
hfinkel wrote:
> If you're checking debug output in a test, you need to add:
> 
>   ; REQUIRES: asserts
> 
> and please do that in a separate test so that we don't lose primary test coverage when compiling without Asserts enabled.
Yes, thanks for saving me from another buildbot crisis :).  Will split the test and add the REQUIRES.

http://reviews.llvm.org/D7691

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list