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

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer 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
 
----------------
Since you have a `ST->hasMVEIntegerOps()` check, do we need a run with `-mattr=-mve`?


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

https://reviews.llvm.org/D88980