[PATCH] D38771: [x86] avoid infinite loop from SoftenFloatOperand (PR34866)

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

In https://reviews.llvm.org/D38771#894855, @chh wrote:

> This change LGTM.
>  I added more comments in https://bugs.llvm.org/show_bug.cgi?id=34866.
>
> Note that fp128-cast.ll tests only cover the cases with +mmx.
>  If the original change in https://reviews.llvm.org/D31156 could affect fp128 and no-mmx cases,
>  it would be better to add test cases for clang -mno-mmx and/or -mno-sse.


Thanks for the explanation. And yes, https://reviews.llvm.org/D31156 has no dependency on mmx. I'll add a no-mmx RUN for this file.


https://reviews.llvm.org/D38771