[PATCH] D127971: [MachinePipeliner] Handle failing constrainRegClass
Dave Green via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Jun 19 10:25:12 PDT 2022
dmgreen added a comment.
In D127971#3590721 <https://reviews.llvm.org/D127971#3590721>, @bcahoon wrote:
> Thanks for making this change. It's an interesting case. We handled a related issue, with subregisters, by adding code to preprocessPhiNodes to create copies. I don't have a problem with approach in this patch. There may be other cases in this code that need a similar change? Not sure...
Thanks for the suggestions. I've looked through and checked the other uses of constrainRegClass, changing them to asserts so that it is more obvious where the call fails.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D127971/new/
https://reviews.llvm.org/D127971
More information about the llvm-commits
mailing list