<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Mar 10, 2014 at 12:28 PM, JF Bastien <span dir="ltr"><<a href="mailto:jfb@chromium.org" target="_blank">jfb@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><p dir="ltr"><br>
> I just realized that the error output is empty, so my proposed change will not work.</p>
</div><p dir="ltr">It works, but it's redundant. I don't really care either way, so I checked in the original patch.</p></blockquote><div>I meant  the following (what I suggested earlier) will not work because t.a.err is empty.</div>
<div><span style="color:rgb(80,0,80);font-family:arial,sans-serif;font-size:13px">> ; RUN: cat %t.a.err | FileCheck --check-prefix=WARN-A %s</span></div><span style="color:rgb(80,0,80);font-family:arial,sans-serif;font-size:13px">> WARN-A-NOT:  WARNING: Linking two modules of different target triples:</span></div>
<br></div><div class="gmail_extra">Your checkin is fine, but I think it is possible to simplify it with</div><div class="gmail_extra"><div class="gmail_extra">; RUN: llvm-link %s %S/Inputs/targettriple-a.ll -S 2>&1 | FileCheck %s --check-prefix=WARN-A</div>
<div class="gmail_extra">; RUN: llvm-link %s %S/Inputs/targettriple-b.ll -S 2>&1 | FileCheck %s --check-prefix=WARN-B<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">target triple = "e"</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">; WARN-A-NOT: WARNING: Linking two modules of different target triples:</div><div class="gmail_extra">; WARN-A: target triple = "e"</div><div class="gmail_extra">
<br></div><div class="gmail_extra">; WARN-B: WARNING: Linking two modules of different target triples:</div><div class="gmail_extra">; WARN-B: target triple = "e"</div></div><div class="gmail_extra"><br></div><div class="gmail_extra">
Cheers,</div><div class="gmail_extra">Manman</div></div>