[PATCH] D82703: [InstCombine] convert assumes to operand bundles

[Tyker via Phabricator via llvm-commits]

Tyker added inline comments.


================
Comment at: llvm/test/Transforms/InstCombine/assume.ll:255
+; BUNDLES-NEXT:    tail call void @escape(i32* nonnull [[LOAD]])
+; BUNDLES-NEXT:    ret i1 false
 ;
----------------
jdoerfert wrote:
> It is questionable if we should prefer the assume or the `!nonnull` metadata here.
fixed


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82703