[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:710
+  ret void
+}
+
----------------
jdoerfert wrote:
> Also unclear if this is better than the GEP. I guess the case with an offset of 4 is clear but this one I don't know.
this is one of the default canonicalization done when building assumes maybe we should keep both.
the idea behind it is to make the knowledge usable by all derived pointers and also minimize the number of assumes but i agree it is not clear its better.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82703