<p dir="ltr">Is it worth/possible to put the wrote at the start to, for symmetry (or is there something else before it in the same string?) - otherwise I would wonder if the difference between the start and end was significant in some way.</p>

<div class="gmail_quote">On May 25, 2014 5:53 AM, "Rafael Espindola" <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: rafael<br>
Date: Sun May 25 07:43:13 2014<br>
New Revision: 209599<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=209599&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=209599&view=rev</a><br>
Log:<br>
Make these CHECKs a bit more strict.<br>
<br>
The " at the end of the line makes sure we matched the entire directive.<br>
<br>
Modified:<br>
    llvm/trunk/test/CodeGen/X86/dllexport-x86_64.ll<br>
    llvm/trunk/test/CodeGen/X86/dllexport.ll<br>
<br>
Modified: llvm/trunk/test/CodeGen/X86/dllexport-x86_64.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/dllexport-x86_64.ll?rev=209599&r1=209598&r2=209599&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/dllexport-x86_64.ll?rev=209599&r1=209598&r2=209599&view=diff</a><br>

==============================================================================<br>
--- llvm/trunk/test/CodeGen/X86/dllexport-x86_64.ll (original)<br>
+++ llvm/trunk/test/CodeGen/X86/dllexport-x86_64.ll Sun May 25 07:43:13 2014<br>
@@ -74,31 +74,31 @@ define weak_odr dllexport void @weak1()<br>
<br>
<br>
 ; CHECK: .section .drectve<br>
-; WIN32: /EXPORT:Var1,DATA<br>
-; WIN32: /EXPORT:Var2,DATA<br>
-; WIN32: /EXPORT:Var3,DATA<br>
-; WIN32: /EXPORT:WeakVar1,DATA<br>
-; WIN32: /EXPORT:WeakVar2,DATA<br>
-; WIN32: /EXPORT:f1<br>
-; WIN32: /EXPORT:f2<br>
-; WIN32: /EXPORT:lnk1<br>
-; WIN32: /EXPORT:lnk2<br>
-; WIN32: /EXPORT:weak1<br>
-; WIN32: /EXPORT:alias<br>
-; WIN32: /EXPORT:alias2<br>
-; WIN32: /EXPORT:alias3<br>
-; WIN32: /EXPORT:weak_alias<br>
-; MINGW: -export:Var1,data<br>
-; MINGW: -export:Var2,data<br>
-; MINGW: -export:Var3,data<br>
-; MINGW: -export:WeakVar1,data<br>
-; MINGW: -export:WeakVar2,data<br>
-; MINGW: -export:f1<br>
-; MINGW: -export:f2<br>
-; MINGW: -export:lnk1<br>
-; MINGW: -export:lnk2<br>
-; MINGW: -export:weak1<br>
-; MINGW: -export:alias<br>
-; MINGW: -export:alias2<br>
-; MINGW: -export:alias3<br>
-; MINGW: -export:weak_alias<br>
+; WIN32: /EXPORT:Var1,DATA"<br>
+; WIN32: /EXPORT:Var2,DATA"<br>
+; WIN32: /EXPORT:Var3,DATA"<br>
+; WIN32: /EXPORT:WeakVar1,DATA"<br>
+; WIN32: /EXPORT:WeakVar2,DATA"<br>
+; WIN32: /EXPORT:f1"<br>
+; WIN32: /EXPORT:f2"<br>
+; WIN32: /EXPORT:lnk1"<br>
+; WIN32: /EXPORT:lnk2"<br>
+; WIN32: /EXPORT:weak1"<br>
+; WIN32: /EXPORT:alias"<br>
+; WIN32: /EXPORT:alias2"<br>
+; WIN32: /EXPORT:alias3"<br>
+; WIN32: /EXPORT:weak_alias"<br>
+; MINGW: -export:Var1,data"<br>
+; MINGW: -export:Var2,data"<br>
+; MINGW: -export:Var3,data"<br>
+; MINGW: -export:WeakVar1,data"<br>
+; MINGW: -export:WeakVar2,data"<br>
+; MINGW: -export:f1"<br>
+; MINGW: -export:f2"<br>
+; MINGW: -export:lnk1"<br>
+; MINGW: -export:lnk2"<br>
+; MINGW: -export:weak1"<br>
+; MINGW: -export:alias"<br>
+; MINGW: -export:alias2"<br>
+; MINGW: -export:alias3"<br>
+; MINGW: -export:weak_alias"<br>
<br>
Modified: llvm/trunk/test/CodeGen/X86/dllexport.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/dllexport.ll?rev=209599&r1=209598&r2=209599&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/dllexport.ll?rev=209599&r1=209598&r2=209599&view=diff</a><br>

==============================================================================<br>
--- llvm/trunk/test/CodeGen/X86/dllexport.ll (original)<br>
+++ llvm/trunk/test/CodeGen/X86/dllexport.ll Sun May 25 07:43:13 2014<br>
@@ -93,38 +93,38 @@ define weak_odr dllexport void @weak1()<br>
<br>
<br>
 ; CHECK: .section .drectve<br>
-; CHECK-CL: /EXPORT:_Var1,DATA<br>
-; CHECK-CL: /EXPORT:_Var2,DATA<br>
-; CHECK-CL: /EXPORT:_Var3,DATA<br>
-; CHECK-CL: /EXPORT:_WeakVar1,DATA<br>
-; CHECK-CL: /EXPORT:_WeakVar2,DATA<br>
-; CHECK-CL: /EXPORT:_f1<br>
-; CHECK-CL: /EXPORT:_f2<br>
-; CHECK-CL: /EXPORT:_stdfun@0<br>
-; CHECK-CL: /EXPORT:@fastfun@0<br>
-; CHECK-CL: /EXPORT:_thisfun<br>
-; CHECK-CL: /EXPORT:_lnk1<br>
-; CHECK-CL: /EXPORT:_lnk2<br>
-; CHECK-CL: /EXPORT:_weak1<br>
-; CHECK-CL: /EXPORT:_alias<br>
-; CHECK-CL: /EXPORT:_alias2<br>
-; CHECK-CL: /EXPORT:_alias3<br>
-; CHECK-CL: /EXPORT:_weak_alias<br>
-; CHECK-GCC: -export:Var1,data<br>
-; CHECK-GCC: -export:Var2,data<br>
-; CHECK-GCC: -export:Var3,data<br>
-; CHECK-GCC: -export:WeakVar1,data<br>
-; CHECK-GCC: -export:WeakVar2,data<br>
-; CHECK-GCC: -export:f1<br>
-; CHECK-GCC: -export:f2<br>
-; CHECK-GCC: -export:stdfun@0<br>
-; CHECK-GCC: -export:@fastfun@0<br>
-; CHECK-GCC: -export:thisfun<br>
-; CHECK-GCC: -export:lnk1<br>
-; CHECK-GCC: -export:lnk2<br>
-; CHECK-GCC: -export:weak1<br>
-; CHECK-GCC: -export:alias<br>
-; CHECK-GCC: -export:alias2<br>
-; CHECK-GCC: -export:alias3<br>
-; CHECK-GCC: -export:weak_alias<br>
+; CHECK-CL: /EXPORT:_Var1,DATA"<br>
+; CHECK-CL: /EXPORT:_Var2,DATA"<br>
+; CHECK-CL: /EXPORT:_Var3,DATA"<br>
+; CHECK-CL: /EXPORT:_WeakVar1,DATA"<br>
+; CHECK-CL: /EXPORT:_WeakVar2,DATA"<br>
+; CHECK-CL: /EXPORT:_f1"<br>
+; CHECK-CL: /EXPORT:_f2"<br>
+; CHECK-CL: /EXPORT:_stdfun@0"<br>
+; CHECK-CL: /EXPORT:@fastfun@0"<br>
+; CHECK-CL: /EXPORT:_thisfun"<br>
+; CHECK-CL: /EXPORT:_lnk1"<br>
+; CHECK-CL: /EXPORT:_lnk2"<br>
+; CHECK-CL: /EXPORT:_weak1"<br>
+; CHECK-CL: /EXPORT:_alias"<br>
+; CHECK-CL: /EXPORT:_alias2"<br>
+; CHECK-CL: /EXPORT:_alias3"<br>
+; CHECK-CL: /EXPORT:_weak_alias"<br>
+; CHECK-GCC: -export:Var1,data"<br>
+; CHECK-GCC: -export:Var2,data"<br>
+; CHECK-GCC: -export:Var3,data"<br>
+; CHECK-GCC: -export:WeakVar1,data"<br>
+; CHECK-GCC: -export:WeakVar2,data"<br>
+; CHECK-GCC: -export:f1"<br>
+; CHECK-GCC: -export:f2"<br>
+; CHECK-GCC: -export:stdfun@0"<br>
+; CHECK-GCC: -export:@fastfun@0"<br>
+; CHECK-GCC: -export:thisfun"<br>
+; CHECK-GCC: -export:lnk1"<br>
+; CHECK-GCC: -export:lnk2"<br>
+; CHECK-GCC: -export:weak1"<br>
+; CHECK-GCC: -export:alias"<br>
+; CHECK-GCC: -export:alias2"<br>
+; CHECK-GCC: -export:alias3"<br>
+; CHECK-GCC: -export:weak_alias"<br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>