[PATCH] D116578: [LegalizeTypes] Remove IsVP argument from type legalization methods. NFC

Fraser Cormack via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 4 04:37:22 PST 2022


frasercrmck accepted this revision.
frasercrmck added a comment.
This revision is now accepted and ready to land.

LGTM. I originally went for `IsVP` as it's arguably clearer to the reader, but I suppose it is better to avoid relying on extra params when the info is already known via other means. The `isVPOpcode` assertions help make sense of it.


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

https://reviews.llvm.org/D116578



More information about the llvm-commits mailing list