[PATCH] D47199: [MC] Remove PhysRegSize from MCRegisterClass

Bjorn Pettersson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 22 08:55:16 PDT 2018


bjope added a comment.

Is this cleanup OK now?
I know that @kparzysz tried this originally in https://reviews.llvm.org/D31299, but that refactoring ended up leaving these methods and the PhysRegSize member around.

It looks like @qcolombet had some opinions about this last time. The original solution for https://reviews.llvm.org/D31299 was reverted in r298727 with the message being "Revert r298652 on Quentin's request.
I'm not sure what the problem was. Is this code actually used from somewhere, or was it just to give out-of-tree clones some time to adapt to the new interface?


Repository:
  rL LLVM

https://reviews.llvm.org/D47199





More information about the llvm-commits mailing list