[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