[PATCH] D69656: [IPCP] Bail on extractvalue's with more than 1 index.

Thu Oct 31 11:40:43 PDT 2019

bjope added a comment.

In D69656#1728773 <https://reviews.llvm.org/D69656#1728773>, @jdoerfert wrote:

> Besides the point: I didn't know anyone was still using this pass. Actually I was going to propose to delete it soon. Is there a desire to keep it or would people switch to IP-SCCP and/or the Attributor (assuming they subsume the functionality)? The one reason to keep this pass is that it is simple and "fast" but it's questionable if that is useful if we run either of the other ones anyway.
>
> The patch LGTM.

I assume that I was added as reviewer due to being the last person who contributed to this pass.
It looks like the fuzzy test framework we use downstream (out-of-tree) sometimes add this pass to the opt pipeline. I blame that on historical reasons, and it is nothing that we really depend on.
So do not see my contribution earlier this year as a reason for keeping the pass around. I have no clue if the reporter of PR43857 is using this pass for something less exotic than fuzzy testing.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69656