[llvm-commits] [llvm] r83391 - /llvm/trunk/lib/Support/Triple.cpp

[Chris Lattner]

On Oct 6, 2009, at 11:07 AM, Collin Winter wrote:

> Hi,
>
> 2009/10/6 Török Edwin <edwintorok at gmail.com>:
>> On 2009-10-06 20:35, Jeffrey Yasskin wrote:
>>> On Tue, Oct 6, 2009 at 10:25 AM, Jeffrey Yasskin <jyasskin at google.com 
>>> > wrote:
>>>>  void Triple::setArchName(const StringRef &Str) {
>>>> -  setTriple(Str + "-" + getVendorName() + "-" +  
>>>> getOSAndEnvironmentName());
>>>> +  // Work around a miscompilation bug in gcc 4.0.3.
>>>>
>>
>> Who uses 4.0.3 still? That a very buggy release.
>
> It's the compiler that shipped with Ubuntu Dapper, which some of our
> project's buildbot slaves are running on. The problem may impact other
> versions of gcc, but since 4.0.3 isn't listed on the "known-bad gcc
> versions", we went ahead and tracked down the problematic code
> snippet.

IMO, if compiler bugs are easy to work-around, it is good to do so.   
That increases the audience that can use llvm without installing other  
random compilers to make it work.

-Chris