Re: [Mesa-dev] [PATCH] gallivm: Replace architecture test with PIPE_ARCH_*

2012-02-10 Thread Jose Fonseca
Looks good. Thanks. Jose - Original Message - > X86Target is a variable, and therefore isn't defined at compile time. > So > LLVM_NATIVE_ARCH == X86Target > is translated into > 0 == 0 > and since X86 is first, we always pick it. > > Therefore we replace the logic with PIPE_ARCH_*. >

[Mesa-dev] [PATCH] gallivm: Replace architecture test with PIPE_ARCH_*

2012-02-09 Thread Stéphane Marchesin
X86Target is a variable, and therefore isn't defined at compile time. So LLVM_NATIVE_ARCH == X86Target is translated into 0 == 0 and since X86 is first, we always pick it. Therefore we replace the logic with PIPE_ARCH_*. https://bugs.freedesktop.org/show_bug.cgi?id=45420 --- src/gallium/auxili