<div dir="ltr">I don't have a better idea.<br><br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature">~Craig</div></div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Feb 14, 2023 at 12:32 PM Alex Bradbury <<a href="mailto:asb@igalia.com">asb@igalia.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2023-02-14 19:56, Craig Topper wrote:<br>
> How does this interact with the CHECK-NOTs above it. Does it check<br>
> that they don't appear before the __riscv_i but would be allowed after<br>
> it?<br>
<br>
Thanks for the post-commit review. I think you're right.<br>
<br>
It looks like the easiest fix is probably move the __riscv_i check out<br>
of that block of CHECK-NOTs and just add a new:<br>
<br>
// RUN: %clang -target riscv32-unknown-linux-gnu -march=rv32i -x c -E<br>
-dM %s \<br>
// RUN: -o - | FileCheck %s<br>
// RUN: %clang -target riscv64-unknown-linux-gnu -march=rv64i -x c -E<br>
-dM %s \<br>
// RUN: -o - | FileCheck %s<br>
// CHECK: __riscv_i 2000000{{$}}<br>
<br>
Do you agree, or am I missing a better alternative?<br>
<br>
Cheers,<br>
<br>
Alex<br>
</blockquote></div>