[PATCH] D41583: [x86][icelake][vaes]

coby via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 26 09:47:48 PST 2017

coby added inline comments.

Comment at: lib/Basic/Targets/X86.cpp:573
     setMMXLevel(Features, AMD3DNowAthlon, Enabled);
   } else if (Name == "aes") {
     if (Enabled)
craig.topper wrote:
> Shouldn't -aes imply -vaes?
how come? perhaps i'm missing here something?
why would the first imply the latter?
following this road an atom z8XXX should be capable of supporting vaes, for example (where it lacks avx, for example, https://www.intel.com/content/www/us/en/processors/atom/atom-z8000-datasheet-vol-1.html)
also, in that sense, pclmul is implying vpclmulqdq

  rC Clang


More information about the cfe-commits mailing list