[llvm-commits] [llvm] r93854 - /llvm/trunk/lib/Target/X86/AsmPrinter/X86AsmPrinter.cpp

[Chris Lattner]

On Jan 20, 2010, at 12:26 PM, Anton Korobeynikov wrote:

> Hi, Chris
>
>> more cleanups.  Emit the .local directive even on cygwin/mingw.
>> I'm not sure that this is correct,
> No, it's not correct. binutils on mingw does not support .local,
> please revert the patch.
>
> Unfortunately, there is no buildbot for mingw (though all necessary
> scripts were sent to Daniel ~week ago) - you'd see the problem
> there....

It turns out that this doesn't even matter at all.  Cygwin  
supports .lcomm so we already use that instead of .comm + .local.

-Chris