[PATCH] D88980: [ARM] Basic getArithmeticReductionCost reduction costs

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/test/Analysis/CostModel/ARM/mve-vecreduce-add.ll:2
 ; NOTE: Assertions have been autogenerated by utils/update_analyze_test_checks.py
 ; RUN: opt < %s -S -mtriple=thumbv8.1m.main-none-eabi -mattr=+mve.fp -cost-model -analyze | FileCheck %s
 
----------------
SjoerdMeijer wrote:
> Since you have a `ST->hasMVEIntegerOps()` check, do we need a run with `-mattr=-mve`?
Do you mean, are there tests without MVE? If so yeah, but they are in a different test file and did not change. reduce-add.ll in the same folder contains some other basic tests for Neon and without any vectorization.


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

https://reviews.llvm.org/D88980