[PATCH] D118917: [AArch64][NFC] Remove call to useSVEForFixedLengthVectors in useSVEForFixedLengthVectorVT

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 3 09:03:26 PST 2022


david-arm added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:3190
 SDValue AArch64TargetLowering::LowerXOR(SDValue Op, SelectionDAG &DAG) const {
-  if (useSVEForFixedLengthVectorVT(Op.getValueType()))
+  if (Subtarget->useSVEForFixedLengthVectors() &
+      useSVEForFixedLengthVectorVT(Op.getValueType()))
----------------
CarolineConcatto wrote:
> Is it missing and extra &?
You're right @CarolineConcatto, good spot!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118917



More information about the llvm-commits mailing list