[PATCH] D124506: [NFC][AArch64][CodeGen] Use ArrayRef in TargetLowering functions

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 27 03:00:18 PDT 2022


paulwalker-arm added a comment.

Can you provide a little detail as to the value of this change. I find it common to grep for `setOperationAction(ISD::blah` to identify the legalisation strategy.  After this change that becomes impossible.  I'm not saying I'm against this change but it would be nice to know what we get back in return for loosing this flexibility.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124506



More information about the llvm-commits mailing list