<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Sep 28, 2015 at 10:35 PM, Sean Silva <span dir="ltr"><<a href="mailto:chisophugis@gmail.com" target="_blank">chisophugis@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">silvas added inline comments.<br>
<span class=""><br>
================<br>
Comment at: test/elf2/invalid-elf.test:1-8<br>
@@ -1,9 +1,1 @@<br>
-RUN: not lld -flavor gnu2 %p/Inputs/invalid-data-encoding.elf -o %t2 2>&1 | \<br>
-RUN:   FileCheck --check-prefix=INVALID-DATA-ENC %s<br>
-INVALID-DATA-ENC: Invalid data encoding: {{.*}}/invalid-data-encoding.elf<br>
-<br>
-RUN: not lld -flavor gnu2 %p/Inputs/invalid-file-class.elf -o %t2 2>&1 | \<br>
-RUN:   FileCheck --check-prefix=INVALID-FILE-CLASS %s<br>
-INVALID-FILE-CLASS: Invalid file class: {{.*}}/invalid-file-class.elf<br>
-<br>
 RUN: not lld -flavor gnu2 %p/Inputs/invalid-symtab-sh_info.elf -o %t2 2>&1 | \<br>
----------------<br>
</span>But you did not remove the place where we print the error message. Should we convert the error messages into assertions?<br></blockquote><div><br></div><div>Good point. We should remove that code. I'll do.</div></div></div></div>