[PATCH] D74436: Change clang option -ffp-model=precise to select ffp-contract=on
Melanie Blower via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Jun 19 05:26:08 PDT 2021
mibintc added a comment.
I’ve been trying to commit this patch. However, when I commit, the tests fail some 21 floating point tests in LNT. Also there is both performance improvement as well as regression which shouldn’t happen because the sum total of my clang patch + my LNT patch should effectively be a no-op.
It appears that the correction I made to LNT is not effective, but I’ve tested the patch on my Linux box and it does work. My patch unconditionally overrides the command line to restore the original LNT behavior. I verified the commit revision to LNT and the bot is running my patched version of LNT. It looks like the bot is using a different rule to build the test cases.
It’s hard to see exactly what the bot is doing, there isn’t enough information. It would be nice if the bot Makefile was built with ‘verbose’ so I could see all the compile commands that the bot is using.
One of the bot failures is here: http://lnt.llvm.org/db_default/v4/nts/147167
I can see the bot owner is Yin Yang, I will reach out.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D74436/new/
https://reviews.llvm.org/D74436
More information about the cfe-commits
mailing list