[PATCH] D127680: [BasicTTI] Allow generic handling of scalable vector fshr/fshl

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:425
+  case Intrinsic::fshr:
+    // Match legacy behavior - this is probably not the right costing here
+    if (isa<ScalableVectorType>(RetTy))
----------------
craig.topper wrote:
> Seems like this should be a FIXME?
I don't think this needs to be added. Just updating the costs in the test files to whatever is now produced will likely be more accurate, and more in-line with the scalar and fixed length vector costs. We can then update them more accurately in the future.


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

https://reviews.llvm.org/D127680