[PATCH] D141068: [AArch64][SME]: Make 'Expand' the default action for all Ops.

[hassnaaHamdi via Phabricator via llvm-commits]

hassnaa-arm added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:4329
 
-  if (useSVEForFixedLengthVectorVT(OpVT))
+  if (useSVEForFixedLengthVectorVT(OpVT,
+                                   Subtarget->forceStreamingCompatibleSVE()))
----------------
sdesmalen wrote:
> hassnaa-arm wrote:
> > sdesmalen wrote:
> > > Can you split out this change, along with marking BITCAST as custom, into a separate patch?
> > Removing BITCAST from this patch will cause a crash to some testing files, is that okay ?
> That is not okay :) It just means you need to reorder the patch so the other one comes first.
I found that I was mistaken about custom-lowering BITCAST. No need for making it custom.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141068