Michael,<div><br></div><div>Thanks for taking the time to get this into the mainline.</div><div><br></div><div> - Nathan<br><br><div class="gmail_quote">On Mon, Jul 26, 2010 at 7:25 PM, Michael Spencer <span dir="ltr"><<a href="mailto:bigcheesegs@gmail.com">bigcheesegs@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Daniel somehow replied to one of my previous commits on llvm-commits<br>
instead of this thread.<br>
<br>
On Mon, Jul 26, 2010 at 12:41 AM, Daniel Dunbar <<a href="mailto:daniel@zuster.org">daniel@zuster.org</a>> wrote:<br>
> Hi Michael,<br>
><br>
> Two minor notes:<br>
> --<br>
>> diff --git a/lib/Target/X86/X86AsmBackend.cpp b/lib/Target/X86/X86AsmBackend.cpp<br>
>> index 2cf65c1..02ac2be 100644<br>
>> --- a/lib/Target/X86/X86AsmBackend.cpp<br>
>> +++ b/lib/Target/X86/X86AsmBackend.cpp<br>
>> @@ -14,6 +14,7 @@<br>
>>  #include "llvm/MC/MCAssembler.h"<br>
>>  #include "llvm/MC/MCExpr.h"<br>
>>  #include "llvm/MC/MCObjectWriter.h"<br>
>> +#include "llvm/MC/MCSectionCOFF.h"<br>
>>  #include "llvm/MC/MCSectionELF.h"<br>
>>  #include "llvm/MC/MCSectionMachO.h"<br>
>>  #include "llvm/MC/MachObjectWriter.h"<br>
>> @@ -212,6 +213,24 @@ public:<br>
>>      : ELFX86AsmBackend(T) {}<br>
>>  };<br>
>><br>
>> +class WindowsX86AsmBackend : public X86AsmBackend {<br>
>> +public:<br>
>> +  WindowsX86AsmBackend(const Target &T)<br>
>> +    : X86AsmBackend(T) {<br>
>> +    HasAbsolutizedSet = true;<br>
>> +    HasScatteredSymbols = true;<br>
><br>
> These probably should be false for Win32.<br>
<br>
What do they do?<br>
<br>
>> diff --git a/test/MC/COFF/dg.exp b/test/MC/COFF/dg.exp<br>
>> new file mode 100644<br>
>> index 0000000..7b7bd4e<br>
>> --- /dev/null<br>
>> +++ b/test/MC/COFF/dg.exp<br>
>> @@ -0,0 +1,5 @@<br>
>> +load_lib llvm.exp<br>
>> +<br>
>> +if { [llvm_supports_target X86] } {<br>
>> +  RunLLVMTests [lsort [glob -nocomplain $srcdir/$subdir/*.{ll}]]<br>
>> +}<br>
><br>
> This should be *.{s}.<br>
<br>
This has to be *.{ll} for now because llvm-mc cannot parse COFF .s files.<br>
<br>
> --<br>
><br>
> Otherwise looks good to me!<br>
><br>
>  - Daniel<br>
<font color="#888888"><br>
- Michael Spencer<br>
</font></blockquote></div><br></div>