[PATCH] D63969: GlobalISel: Try to propagate the register bank in LegalizerHelper
Amara Emerson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Jun 30 00:26:28 PDT 2019
aemerson added a comment.
This change makes a pretty significant implicit design change that I don't agree with. It doesn't seem right that the legalizer routines should have to know about regbanks or how to propagate them. I think the two passes should remain separate, an opt-in change observer seems like the better approach here.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63969/new/
https://reviews.llvm.org/D63969
More information about the llvm-commits
mailing list