[PATCH] D128159: [DAG] Enable scalable vectors handling in computeKnownBits

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 20 01:23:44 PDT 2022


RKSimon added a comment.

SGTM but I'll defer to people who are actively working with scalable vectors whether the getVectorMinNumElements approach will work.

I had been wondering whether we're going have to move from APInt to some kind of 'DemandedMask' wrapper class to handle everything,



================
Comment at: llvm/lib/Target/AArch64/AArch64InstrFormats.td:61
+  }];
+}
+
----------------
Just move this as a pre-commit NFC?


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

https://reviews.llvm.org/D128159



More information about the llvm-commits mailing list