[PATCH] D103316: Hoist llvm.assume into single predecessor if block otherwise empty

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

In D103316#2789027 <https://reviews.llvm.org/D103316#2789027>, @markus wrote:

> Wrt assume operand bundles I am thinking that those can always simply be dropped.

I think you can simply ignore operand bundle forms.  This is not on by default, and the implementation isn't yet mature.  The folks working on that can pay the cost of figuring out what to do for them.  As long as your code doesn't crash on operand bundle assumes, I don't really care what it does with them.


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

https://reviews.llvm.org/D103316