RKSimon added inline comments.

================
Comment at: llvm/test/CodeGen/X86/vector-reduce-fmax-nnan.ll:7
 ; RUN: llc < %s -mtriple=x86_64-unknown-unknown -mattr=+avx512f,+avx512bw | 
FileCheck %s --check-prefixes=ALL,AVX512
 ; RUN: llc < %s -mtriple=x86_64-unknown-unknown 
-mattr=+avx512f,+avx512bw,+avx512vl | FileCheck %s --check-prefixes=ALL,AVX512
+; RUN: llc < %s -mtriple=x86_64-unknown-unknown -mattr=+avx512fp16,+avx512vl | 
FileCheck %s --check-prefixes=ALL,AVX512,AVX512FP16
----------------
I think you need to add a fallback prefix for AVX512 without FP16


================
Comment at: llvm/test/CodeGen/X86/vector-reduce-fmin-nnan.ll:8
 ; RUN: llc < %s -mtriple=x86_64-unknown-unknown 
-mattr=+avx512f,+avx512bw,+avx512vl | FileCheck %s --check-prefixes=ALL,AVX512
+; RUN: llc < %s -mtriple=x86_64-unknown-unknown -mattr=+avx512fp16,+avx512vl | 
FileCheck %s --check-prefixes=ALL,AVX512,AVX512FP16
 
----------------
I think you need to add a fallback prefix for AVX512 without FP16


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105264

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to