[PATCH] D46242: Support a funclet operand bundle in LowerInvoke

[Heejin Ahn via Phabricator via llvm-commits]

aheejin added a comment.

@rnk Ping? Thanks for your comments! I understand that I need to preserve other non-funclet operand bundles, and will do that, but what I'm not sure about is, as I said, if we are supposed to remove "funclet" bundles in this pass, should we do the same thing for "funclet" bundles attached to call instructions for consistency? If we should, would it make sense to be a part of "LowerInvoke" pass, the name that sounds like suggesting it only touches invoke instructions?


Repository:
  rL LLVM

https://reviews.llvm.org/D46242