[llvm-commits] PATCH: Fix for PR6679

Benjamin Kramer benny.kra at googlemail.com
Fri Apr 27 05:09:57 PDT 2012


On 17.04.2012, at 06:44, Christoph Erhardt wrote:

> Hi,
> 
> attached is an updated patch for bug 6679. The original patch missed one
> case where a FUCOMI* instruction could still be selected despite being
> unsupported by the subtarget.
> 
> By the way, Clang appears to miscompile FFTW 3.3.1, resulting in a
> reproducible segfault. This has nothing to do with bug 6679 or my
> proposed fix per se, but hit me when doing some testing. The issue is
> limited to 32-bit code (-m32) at optimization levels of -O1 or higher
> and can be reproduced with 3.0, trunk and Xcode 4.3.2 on Linux and OSX.
> I'll open a bug report once I find the time to dig into this.

Looks good, applied as r155704. Thanks for caring about old CPUs :)

- Ben



More information about the llvm-commits mailing list