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

Jeroen Dobbelaere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 19 08:04:16 PST 2021


jeroen.dobbelaere added inline comments.
Herald added a subscriber: dexonsmith.


================
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
+
----------------
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 ?




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