[PATCH] D26593: [X86][GlobalISel] Add minimal call lowering support to the IRTranslator

[Zvi Rackover via llvm-commits]

zvi added a comment.

In https://reviews.llvm.org/D26593#593924, @davide wrote:

>




> OK, I apologize if I missed the whole plan. So, is the goal that of growing this prototype into something ready for production? I remember there were some aspects that, although valuable, were voluntarily ignored for the prototype timeframe because there wasn't enough time (e.g. reuse of instcombine for the new selector).
>  I'm pretty sure there are others (which I don't have off the top of my head).
>  Are these aspects being reconsidered now (out of curiosity)?

No need to apologize - you should feel free to raise your concerns.
The questions in your last comment are about GlobalISel in general which I would prefer the core development team (which are also the reviewers) answer them. Perhaps open a thread in llvm-dev?


Repository:
  rL LLVM

https://reviews.llvm.org/D26593