This is an archive of the discontinued LLVM Phabricator instance.

Add flang-x86_64-windows builder.
ClosedPublic

Authored by Meinersbur on May 26 2021, 12:58 PM.

Details

Summary

Add a buildbot to compile flang using msvc. Due to msvc doing some things differently than other compilers, the msvc flang build breaks regularly. This builder will notify committers if their change causes the build to break.

However, the tests never completed successfully on Windows, partially because many tests require a UNIX shell (see D89368). To not constantly fail, tests failures only trigger a warning, until all tests are fixed as well.

Currently running here: http://meinersbur.de:8011/#/builders/146

Diff Detail

Event Timeline

Meinersbur created this revision.May 26 2021, 12:58 PM
Meinersbur requested review of this revision.May 26 2021, 12:58 PM

Frankly, I don't like the idea of allowing failing tests, definitely not in production. I see what you want to achieve with this, though.

How about not running tests in production till they are working properly, and have them on in the staging? You will have about the same result - test failures will not be reported nor annotated in github, but those who are interested could see the failures, analyze the logs, and reference particular builds. Once all the failing tests are sorted out and passing, they will be enabled in the production. You would have to have at least 2 workers for this setup. Would this do?

buildbot/osuosl/master/config/builders.py
1621

Could you skip the clean arg to use the default, please?

  • Remove clean=False which is the default anyway.

How about not running tests in production till they are working properly, and have them on in the staging? You will have about the same result - test failures will not be reported nor annotated in github, but those who are interested could see the failures, analyze the logs, and reference particular builds. Once all the failing tests are sorted out and passing, they will be enabled in the production. You would have to have at least 2 workers for this setup. Would this do?

While I like the idea, I only have one system for this. The most I could do is some scripting that alternate between two worker processes.

Meinersbur marked an inline comment as done.
  • Remove allow_test_fail

Windows flang tests are now green after D104011 and D89368. allow_test_fail not required anymore.
See http://meinersbur.de:8011/#/builders/146/builds/593

This revision is now accepted and ready to land.Jun 17 2021, 12:25 PM
This revision was automatically updated to reflect the committed changes.