[PATCH] D143287: [Clang][X86] Change X86 cast intrinsics to use __builtin_nondeterministic_value

Juneyoung Lee via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 8 14:21:57 PST 2023


aqjune added a comment.

In D143287#4179344 <https://reviews.llvm.org/D143287#4179344>, @ManuelJBrito wrote:

> In D143287#4179020 <https://reviews.llvm.org/D143287#4179020>, @aqjune wrote:
>
>> H, is D104790 <https://reviews.llvm.org/D104790> superseded by this patch?
>
> I don't think so we still need to fix the undefined intrinsics, right? Maybe I'm not understanding the question.

Oh right, D104790 <https://reviews.llvm.org/D104790> and this deal with different intrinsics, thanks.
I was wondering whether I could clean up some of my open patches.

>> I wonder what is the status of this patch as well.
>
> We need to land D144903 <https://reviews.llvm.org/D144903> first to have the correct assembly, but it's currently held up by a crash in one of the tests due to an unrelated issue.

I see, hope that the crash is fixed soon..!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143287



More information about the cfe-commits mailing list