[PATCH] D93769: [clang] Add support for option -ffp-eval-method and extend #pragma float_control similarly

2021-08-27 Thread Matvey Larionov via Phabricator via cfe-commits
matthewtff added a comment. This CL breaks -frewrite-includes flow. I've create a repro: https://pastebin.com/fFuUdsfp If you buid it on linux with ToT clang like this: clang++ -target i386-unknown-linux-gnu repro.cc -o repro.bin and then run the binay you'll get output "Hello, One". But if you

[PATCH] D93769: [clang] Add support for option -ffp-eval-method and extend #pragma float_control similarly

2021-08-30 Thread Matvey Larionov via Phabricator via cfe-commits
matthewtff added a comment. > It looks like Melanie asked the questions but it wasn't resolved. Can this CL be reverted and relanded later with a fix? This breakage is crucial for a lot of remote/distributed compilation systems. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D93769: [clang] Add support for option -ffp-eval-method and extend #pragma float_control similarly

2021-08-30 Thread Matvey Larionov via Phabricator via cfe-commits
matthewtff added a comment. > Did someone perhaps fix this in trunk? I don't see it on godbolt: > https://godbolt.org/z/r76rx5Evd You've missed the -target argument. It still reproduces with correct arguments: https://godbolt.org/z/1ozhjYvhf Repository: rG LLVM Github Monorepo CHANGES SIN