[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