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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to