[PATCH] D132477: Improve cost model for some 128-bit vector operations that use SVE

[Hassnaa Hamdi via Phabricator via llvm-commits]

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


================
Comment at: llvm/test/Analysis/CostModel/AArch64/sve-fixed-length-div-mul.ll:1
+; NOTE: Assertions have been autogenerated by utils/update_analyze_test_checks.py
+; RUN: opt < %s -passes='print<cost-model>' 2>&1 -disable-output -mcpu=neoverse-v1  -mtriple=aarch64-linux-gnu -mattr=+sve | FileCheck %s
----------------
david-arm wrote:
> I think it might be worth moving the fixed-width vector tests into the existing file:
> 
>   test/Analysis/CostModel/AArch64/sve-fixed-length.ll
> 
> Then you could rename this file to just be something like
> 
>   test/Analysis/CostModel/AArch64/sve-arith.ll
on the main branch, when I run the cost model test for this file:
test/Analysis/CostModel/AArch64/sve-fixed-length.ll, **it gets changed. **


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132477