A missing O3 flag in the VLS and VLA buildbots was resulting
in some false positives slipping through. e.g., CLAMR fails when
running LNT with O3 but passes without it.
Details
Diff Detail
- Repository
- rZORG LLVM Github Zorg
Event Timeline
This looks right to me but please allow time to pass for other reviewers to chime in before submitting.
Hi @MattDevereau ,
Thanks.
Could you elaborate in the commit log about the false positives?
Does it make sense to add -O3 to other bots?
@maxim-kuvyrkov I've expanded the commit message a bit.
AFAIK these 4 bot configs were created on request of ARM specifically for testing VLA/VLS with SVE enabled machines. Changing other bots might interfere with other peoples work so its probably best to keep it simple here.
I've expanded the commit message a bit.
If you're using arc it won't update the commit message on Phab for you, you have to do that manually.
By the way you can update the commit message in Phabricator to match what you have locally with arc diff --verbatim.
So far as I can tell any commit message clarification appears to not have made it into Git or Phabriactor(?). So to clarify, the "false positives" Matt refers to are that we encountered LNT failures due to SVE VLS code-generation where Buildbot was green. As far as we can tell, the reason is that Buildbot was not exercising vector codegen at the default optimization level, and therefore, was of limited use as a VLS builder.
By the way you can update the commit message in Phabricator to match what you have locally with arc diff --verbatim.
TIL this exists. Has always bugged me.
Matt refers to are that we encountered LNT failures due to SVE VLS code-generation where Buildbot was green.
Got it, somehow I thought that you were seeing failures on the bot that I wasn't. But yeah makes total sense, the bot wasn't generating any vector code but your testing elsewhere was.
If tests require -O3 shouldn’t those tests enforce the right flags in the RUN instead? Sounds like an issue with the tests in question.
I don’t mind running these builders with -O3, though.