mibintc added a comment. In D53157#1525311 <https://reviews.llvm.org/D53157#1525311>, @kpn wrote:
> Oh, this ticket is not going to die from neglect. It is true that D43515 > <https://reviews.llvm.org/D43515> is a higher priority, but I need this > IRBuilder work done as well. My department head wanted it done by the end of > _last_ year. It's not going to die. > > How about I merge your changes into this ticket and we continue work over > here? There is the issue of the documentation that lebedev.ri asked you to > write. Can I talk you into putting that together and sending it to me > <kevin.n...@sas.com>? I'll work on the documentation that I was asked to > write. Between the two of us we should be in pretty good shape. Does that > work for you? Yes I'll do that. Thanks. > I am still waiting for feedback from an actual consumer of the IRBuilder who > will be using this new functionality. If someone clang-side could chime in on > this ticket I'd very much appreciate it. I wrote a clang patch that works with your IRBuilder modifications. I also checked with the Intel fortran team and they think this interface will be workable for them too. From Intel perspective it's +1. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D53157/new/ https://reviews.llvm.org/D53157 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
