<div dir="ltr"><div dir="ltr"><div dir="ltr">On Mon, Feb 11, 2019 at 11:38 AM Aaron Ballman via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, Feb 11, 2019 at 11:36 AM Nico Weber via cfe-commits<br>
<<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br>
><br>
> Author: nico<br>
> Date: Mon Feb 11 08:37:02 2019<br>
> New Revision: 353718<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=353718&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=353718&view=rev</a><br>
> Log:<br>
> Make test actually test something (colons were missing)<br>
<br>
Good catch! I wonder if we could/should automatically catch this issue<br>
with a clang-tidy check of some kind?<br></blockquote><div><br></div><div>I sent <a href="https://reviews.llvm.org/D58061">https://reviews.llvm.org/D58061</a> to clean up other instances I could fine. Coming up with that regex took a few iterations.</div><div><br></div><div>I thin the Best Fix is probably to make FileCheck diag if a line starts with (after a few whitelisted comment chars like // and #) a check-prefix but then isn't followed by : (maybe after -NOT, -SAME, -LABEL etc).</div><div><br></div><div>Looks like this specific change here doesn't work on a bot even though it worked locally, looking at that now, sigh.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
~Aaron<br>
<br>
><br>
> Modified:<br>
>     cfe/trunk/test/CodeGen/ms-x86-intrinsics.c<br>
><br>
> Modified: cfe/trunk/test/CodeGen/ms-x86-intrinsics.c<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGen/ms-x86-intrinsics.c?rev=353718&r1=353717&r2=353718&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGen/ms-x86-intrinsics.c?rev=353718&r1=353717&r2=353718&view=diff</a><br>
> ==============================================================================<br>
> --- cfe/trunk/test/CodeGen/ms-x86-intrinsics.c (original)<br>
> +++ cfe/trunk/test/CodeGen/ms-x86-intrinsics.c Mon Feb 11 08:37:02 2019<br>
> @@ -144,28 +144,28 @@ unsigned __int64 test__shiftleft128(unsi<br>
>    return __shiftleft128(l, h, d);<br>
>  }<br>
>  // CHECK-X64-LABEL: define dso_local i64 @test__shiftleft128(i64 %l, i64 %h, i8 %d)<br>
> -// CHECK-X64  = zext i64 %h to i128<br>
> -// CHECK-X64  = shl nuw i128 %0, 64<br>
> -// CHECK-X64  = zext i64 %l to i128<br>
> -// CHECK-X64  = or i128 %1, %2<br>
> -// CHECK-X64  = and i8 %d, 63<br>
> -// CHECK-X64  = shl i128 %<br>
> -// CHECK-X64  = lshr i128 %<br>
> -// CHECK-X64  = trunc i128 %<br>
> -// CHECK-X64  ret i64 %<br>
> +// CHECK-X64:  = zext i64 %h to i128<br>
> +// CHECK-X64:  = shl nuw i128 %0, 64<br>
> +// CHECK-X64:  = zext i64 %l to i128<br>
> +// CHECK-X64:  = or i128 %1, %2<br>
> +// CHECK-X64:  = and i8 %d, 63<br>
> +// CHECK-X64:  = shl i128 %<br>
> +// CHECK-X64:  = lshr i128 %<br>
> +// CHECK-X64:  = trunc i128 %<br>
> +// CHECK-X64:  ret i64 %<br>
><br>
>  unsigned __int64 test__shiftright128(unsigned __int64 l, unsigned __int64 h,<br>
>                                       unsigned char d) {<br>
>    return __shiftright128(l, h, d);<br>
>  }<br>
>  // CHECK-X64-LABEL: define dso_local i64 @test__shiftright128(i64 %l, i64 %h, i8 %d)<br>
> -// CHECK-X64  = zext i64 %h to i128<br>
> -// CHECK-X64  = shl nuw i128 %<br>
> -// CHECK-X64  = zext i64 %l to i128<br>
> -// CHECK-X64  = or i128 %<br>
> -// CHECK-X64  = and i8 %d, 63<br>
> -// CHECK-X64  = lshr i128 %<br>
> -// CHECK-X64  = trunc i128 %<br>
> -// CHECK-X64  ret i64 %<br>
> +// CHECK-X64:  = zext i64 %h to i128<br>
> +// CHECK-X64:  = shl nuw i128 %<br>
> +// CHECK-X64:  = zext i64 %l to i128<br>
> +// CHECK-X64:  = or i128 %<br>
> +// CHECK-X64:  = and i8 %d, 63<br>
> +// CHECK-X64:  = lshr i128 %<br>
> +// CHECK-X64:  = trunc i128 %<br>
> +// CHECK-X64:  ret i64 %<br>
><br>
>  #endif // defined(__x86_64__)<br>
><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
> <a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</blockquote></div></div></div>