[llvm] r180992 - Unbreaking the non-x86 build bots by protecting the AVX test code properly.

Aaron Ballman aaron at aaronballman.com
Tue May 7 05:29:30 PDT 2013


On Tue, May 7, 2013 at 2:15 AM, Kuperstein, Michael M
<michael.m.kuperstein at intel.com> wrote:
> Hi Aaron,
>
> This breaks AVX detection 64-bit platforms.
> I can add "|| defined(__x86_64__) || defined(_M_AMD64) || defined (_M_X64)" (this is what Hosts.cpp has), but I just want to make sure this is an omission and not a conscious decision first.

It was an omission -- I was thinking that because it was the
X86SubTarget file, it only applied to x86 code and not x64.  Oops!
Good catch!

~Aaron



More information about the llvm-commits mailing list