[PATCH] D75825: [AssumeBundles] Preserve Information in the inliner

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 19 08:07:08 PST 2021


jdoerfert added inline comments.


================
Comment at: llvm/test/Transforms/Inline/noalias2.ll:3
 ; RUN: opt -inline -enable-noalias-to-md-conversion -S < %s | FileCheck %s
+; RUN: opt -inline -enable-noalias-to-md-conversion --enable-knowledge-retention -S < %s | FileCheck %s --check-prefix=ASSUME
+
----------------
jeroen.dobbelaere wrote:
> jdoerfert wrote:
> > Same as above.
> @jdoerfert, @ Tyker  This test is partly bogus. The 'ASUME' checks will never be triggered. In D75828, the USE_ASSUME is also replaced by ASSUME, but the driver is not modified. Can I just omit the ASUME part ?
> 
> 
Yeah, you should be able to rerun `utils/update_test_checks.py` after removing "check lines".


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75825



More information about the llvm-commits mailing list