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

Mon Nov 30 21:51:43 PST 2020

pengfei accepted this revision.
pengfei added a comment.
This revision is now accepted and ready to land.

LGTM.

================
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
----------------
I think you can use `--check-prefixes=AVX,AVX1` to reduce some duplicated checks. The same to fminnum.ll

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

https://reviews.llvm.org/D92337