[PATCH] Remove duplicate type definitions from f16cintrin.h

John Thompson john.thompson.jtsoftware at gmail.com
Tue May 26 12:26:00 PDT 2015


If we want individual headers to be independent, we probably need a new header called something like. _intrinsic_types.h, as there are a few definitions like __m256, __m256d, and __m256i scattered among multiple headers.  But actually, it seems to be the case that headers like f16cintrin.h are private headers under x86intrin.h, as indicated in the #error directive (#error "Never use <f16cintrin.h> directly; include <x86intrin.h> instead."), hence the need to be independent could be considered relaxed.  If gnu is like this, it might be a good reason not to stray too far.

What do you think?


http://reviews.llvm.org/D9557

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list