[PATCH] D92337: [x86] adjust cost model values for minnum/maxnum with fast-math-flags

Tue Dec 1 07:08:41 PST 2020

spatel marked an inline comment as done.
spatel added inline comments.

================
Comment at: llvm/test/Analysis/CostModel/X86/fmaxnum.ll:4
 ; RUN: opt < %s -cost-model -analyze -mtriple=x86_64-- -mattr=+sse4.2 | FileCheck %s --check-prefixes=SSE
 ; RUN: opt < %s -cost-model -analyze -mtriple=x86_64-- -mattr=+avx | FileCheck %s --check-prefixes=AVX1
 ; RUN: opt < %s -cost-model -analyze -mtriple=x86_64-- -mattr=+avx2 | FileCheck %s --check-prefixes=AVX2
----------------
pengfei wrote:
> I think you can use `--check-prefixes=AVX,AVX1` to reduce some duplicated checks. The same to fminnum.ll
Thanks! I'll update the RUN lines on commit.

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

https://reviews.llvm.org/D92337