jdoerfert added a comment.

In D101849#2766411 <https://reviews.llvm.org/D101849#2766411>, @mikerice wrote:

> In D101849#2764825 <https://reviews.llvm.org/D101849#2764825>, @ABataev wrote:
>
>> No, it is not required. Most probably, needed to simplify test checks, 
>> nothing else.
>
> Thanks.  I'd like to remove the "REQUIRES: x86-registered-target", the -O1 
> for CHECK6,10, and regenerate the CHECK lines.  Unfortunately I am seeing 
> this mangling issue (https://bugs.llvm.org/show_bug.cgi?id=49767) when 
> running the script. @ggeorgakoudis, how did you get past this to generate the 
> CHECK lines in your change?

The workaround is to check if the assertion would trigger and just return from 
the function instead.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101849/new/

https://reviews.llvm.org/D101849

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to