[PATCH] D96174: [SelectionDAG][AArch64] Restrict matchUnaryPredicate to only handle SPLAT_VECTOR for scalable vectors.

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 11 05:18:49 PST 2021


david-arm added a comment.

Hi, yeah I agree with @frasercrmck here. The only reason I brought this up originally is because callers of the function do something like this:

  if (VT.isFixedLengthVector()) {
  ... Do something with BUILD_VECTOR ...
  } else if (VT.isScalableVector()) {
  .. Do something with SPLAT_VECTOR ...
  }

It felt like since we're already doing this an alternative solution might be to check the opcode instead of the vector type. The patch you have right now does work and wouldn't want to hold it up the patch if you don't have time to fix it right now. It was just a suggestion for another way of dealing with the problem.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96174



More information about the llvm-commits mailing list