[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.


The previous code wasn't correct either though: emitting a .comm  
directive won't make it private to a translation unit?

-Chris