[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