[PATCH] D153770: Fix test regression on 32-bit x86.

Michał Górny via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 26 08:48:57 PDT 2023


mgorny added a comment.

In D153770#4449154 <https://reviews.llvm.org/D153770#4449154>, @zahiraam wrote:

> In D153770#4449078 <https://reviews.llvm.org/D153770#4449078>, @mgorny wrote:
>
>> I think you also need to require `X86` target but I'm not 100% sure.
>
> I don't think so. That warning was legal and is related to the option ffp-eval-method. I could have added run lines in the LIT test to expect the warnings but that's not the point of the LIT tests here.

What I meant is that I think clang might not let you use x86_64 triple without X86 target but I could be wrong.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153770



More information about the cfe-commits mailing list