[PATCH] Enable fma and f16c features when avx512f is enabled.

Ahmed Bougacha ahmed.bougacha at gmail.com
Mon Jun 22 14:59:34 PDT 2015


Hi craig.topper,

Since, according to the ISA extension manual, (EVEX versions of) the FMA3/F16C instructions are a core part of AVX512F, it seems to me we should enable the features when avx512f is enabled (e.g., -mavx512f).  If not, users need to either specify a specific CPU, or add the redundant -mfma/-mf16c.

http://reviews.llvm.org/D10617

Files:
  lib/Basic/Targets.cpp
  test/Preprocessor/x86_target_features.c

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D10617.28164.patch
Type: text/x-patch
Size: 4035 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150622/f9fa5688/attachment.bin>


More information about the cfe-commits mailing list