[PATCH] D92341: [NFC][MC] TargetRegisterInfo::getSubReg is a MCRegister.

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 2 15:36:14 PST 2020


mtrofin marked 4 inline comments as done.
mtrofin added a comment.

In D92341#2428982 <https://reviews.llvm.org/D92341#2428982>, @dsanders wrote:

> LGTM with @gjain's suggestions. I think SIRegisterInfo.cpp could also propagate MCRegister a bit further if you wanted but I didn't check all the users of SubReg

Ack - that probably needs its own patch, it gets a bit involved. I'll do it next.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D92341/new/

https://reviews.llvm.org/D92341



More information about the llvm-commits mailing list