<div dir="ltr">Peter,<div><br></div><div>Thank you for the patch. I applied that to my local copy. Will upload a new patch.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 26, 2015 at 6:47 PM, Peter Collingbourne <span dir="ltr"><<a href="mailto:peter@pcc.me.uk" target="_blank">peter@pcc.me.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Clang build fixes: F533032: clang.diff <<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_F533032&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=z_7FRYAssz8VB-zYjnZaG71OzzWd8waUMkl652FKgKk&s=JCB2Yr57UMJXfj8Fi0S02zpI5jk5aH1mOJLnDuSG1nI&e=" target="_blank">http://reviews.llvm.org/F533032</a>><br>
<br>
<br>
================<br>
Comment at: COFF/InputFiles.cpp:242<br>
@@ +241,3 @@<br>
+<br>
+  uint16_t TypeInfo = read16le(Buf + offsetof(ImportHeader, TypeInfo));<br>
+  int Type = TypeInfo & 0x3;<br>
----------------<br>
Here (and in a few other places) you are making assumptions about the host system's data layout. This may be a largely theoretical problem though.<br></blockquote><div><br></div><div>Updated to use coff_import_header instead of ImportHeader.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
================<br>
Comment at: COFF/SymbolTable.cpp:88<br>
@@ +87,3 @@<br>
+      if (!isa<Defined>(Sym->Body)) {<br>
+        llvm::errs() << "undefined symbol: " << Undef->getName() << "\n";<br>
+        Ret = true;<br>
----------------<br>
Should this diagnose that `Undef` is undefined because `Sym->Body` is undefined?<br></blockquote><div><br></div><div>Yes. Added a comment.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
================<br>
Comment at: test/COFF/driver.test:3<br>
@@ +2,3 @@<br>
+# RUN: FileCheck -check-prefix=MISSING %s < %t.log<br>
+MISSING: nosuchfile.obj: no such file or directory<br>
+<br>
----------------<br>
On Linux this reads "nosuchfile.obj: No such file or directory" (capital N).<br>
<div class="HOEnZb"><div class="h5"></div></div></blockquote></div><br></div><div class="gmail_extra">Fixed.</div></div>