[PATCH] D64665: [MachinePipeliner] Refactor schedule emission logic
James Molloy via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 12 14:00:19 PDT 2019
Thanks Matt, but I'm not sure I understand. Is Register a new API that I've
missed landing? do you have a quick pointer to what you're referring to?
Thanks!
JAmes
On Fri, 12 Jul 2019 at 21:58, Matt Arsenault via Phabricator via
llvm-commits <llvm-commits at lists.llvm.org> wrote:
> arsenm added inline comments.
>
>
> ================
> Comment at: lib/CodeGen/MachinePipeliner.cpp:3277-3278
> + for (auto &MO : NewMI->defs()) {
> + unsigned OrigReg = MO.getReg();
> + unsigned DestReg =
> MRI.createVirtualRegister(MRI.getRegClass(OrigReg));
> + NewMI->substituteRegister(OrigReg, DestReg, MO.getSubReg(),
> ----------------
> Should update to use Register
>
>
> Repository:
> rL LLVM
>
> CHANGES SINCE LAST ACTION
> https://reviews.llvm.org/D64665/new/
>
> https://reviews.llvm.org/D64665
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190712/46e28925/attachment.html>
More information about the llvm-commits
mailing list