[PATCH] D129141: [ARM] Switch the costs of mve1beat and mve4beat

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/test/Transforms/LoopVectorize/ARM/pointer_iv.ll:2
 ; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
-; RUN: opt -loop-vectorize -S -mtriple=thumbv8.1m.main-none-none-eabi -mattr=+mve.fp,mve1beat -dce -instcombine --simplifycfg -simplifycfg-require-and-preserve-domtree=1 -tail-predication=disabled < %s | FileCheck %s
+; RUN: opt -loop-vectorize -S -mtriple=thumbv8.1m.main-none-none-eabi -mattr=+mve.fp,mve4beat -dce -instcombine --simplifycfg -simplifycfg-require-and-preserve-domtree=1 -tail-predication=disabled < %s | FileCheck %s
 
----------------
samtebbs wrote:
> Is this required, considering it doesn't change the rest of the test?
It is needed to keep the test checking the same thing. It requires that the MVE instructions have a cost of 1 for the same vectorization to happen.


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

https://reviews.llvm.org/D129141