[PATCH] D100788: [SystemZ] Support i128 inline asm operands

[Ulrich Weigand via Phabricator via llvm-commits]

uweigand added a comment.

This already looks a lot simpler.   I'm now wondering if it would be even better to change the getNumRegisters interface from (ValueVT, RC) to (ValueVT, RegisterVT).   (Or I guess an Optional<RegisterVT> if we don't want to change all callers.)   All callers of getNumRegisters already have the RegisterVT immediately available, and it allows the back end to make any distinction it needs just as well as a RC would.


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

https://reviews.llvm.org/D100788