[PATCH] D86432: [SVE] Fix TypeSize related warnings with IR truncates of scalable vectors

[David Sherwood via Phabricator via llvm-commits]

david-arm marked an inline comment as done.
david-arm added inline comments.


================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfoImpl.h:456
+      if (!DstSize.isScalable() && DL.isLegalInteger(DstSize.getFixedSize()))
         return 0;
       break;
----------------
fpetrogalli wrote:
> How comes that the test that uses scalable vectors on `trunc` is reporting zero? afaict, if we enter this methods with `trunc of scalable vectors` the cost reported by the function should be 1, no?
Hi, the cost is actually then recalculated in BasicTTIImpl::getCastInstrCost if the cost != 0.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86432