<div dir="ltr">Actually, wait - the point of that NOT .loc is to check for any .loc directive, which has arguments and stuff, so .loc{{$}} is too restrictive and wouldn't catch other .locs<br><br>maybe CHECK-NOT:  .loc{{ }}<br><br>to force the space? Does that work?</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 12, 2015 at 2:39 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I think {{$}} is a more idiomatic way to check for end of line (or at least it is in that file) - I'm surprised \n works there, actually... but good to know I guess.<br><br>Otherwise looks fine, please commit.</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 12, 2015 at 2:23 PM, Will Schmidt <span dir="ltr"><<a href="mailto:willschm@us.ibm.com" target="_blank">willschm@us.ibm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi wschmidt, dblaikie,<br>
<br>
The ppc64le platform will emit a .localentry directive.  this is triggering a false-positive against a CHECK-NOT: .loc  in multiline.ll as seen at<br>
<a href="http://lab.llvm.org:8011/builders/ppc64le-clang/builds/1" target="_blank">http://lab.llvm.org:8011/builders/ppc64le-clang/builds/1</a><br>
<br>
<...><br>
<stdin>:31:2: error: CHECK-NOT: string occurred!<br>
 .localentry f2, .Ltmp1-.Ltmp0<br>
<br>
/home/buildbots/ppc64le-clang/clang-ppc64le-1/llvm.src/test/DebugInfo/multiline.ll:17:14: note: CHECK-NOT: pattern specified here<br>
; CHECK-NOT: .loc<br>
<br>
<br>
I didn't see a clear way to call out ".localentry is OK" via check-not, so the change here is to add a simple "\n" to the check-not line.<br>
<br>
REPOSITORY<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D6935" target="_blank">http://reviews.llvm.org/D6935</a><br>
<br>
Files:<br>
  test/DebugInfo/multiline.ll<br>
<br>
Index: test/DebugInfo/multiline.ll<br>
===================================================================<br>
--- test/DebugInfo/multiline.ll<br>
+++ test/DebugInfo/multiline.ll<br>
@@ -14,7 +14,7 @@<br>
<br>
<br>
 ; CHECK: .loc  1 2 0{{$}}<br>
-; CHECK-NOT: .loc<br>
+; CHECK-NOT: .loc\n<br>
 ; CHECK: .loc  1 3 3 prologue_end{{$}}<br>
 ; CHECK-NOT: .loc<br>
 ; CHECK: .loc  1 3 9 is_stmt 0{{$}}<br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>