This is an archive of the discontinued LLVM Phabricator instance.

[LLDB] Port toolchain-msvc.test for Arm/AArch4 Windows
ClosedPublic

Authored by omjavaid on Jan 19 2022, 6:43 AM.

Details

Summary

This patch updates toolchain-msvc.test to cater for Arm64 windows platform.

Diff Detail

Event Timeline

omjavaid created this revision.Jan 19 2022, 6:43 AM
omjavaid requested review of this revision.Jan 19 2022, 6:43 AM
labath added a subscriber: labath.Jan 19 2022, 6:56 AM

FWIW, I consider the build.py script a failed experiment. The idea itself was not completely bad, but the lack of buy-in from sufficient amount of developers meant that it never really had a chance of fulfilling its (rather lofty) goals.

I think it should be removed or at least scaled back significantly. The only thing it's really used for these days is to invoke msvc and clang-cl, and I'm sure we can achieve the same thing with some lit substitutions as well...

To put it in a more constructive light, what I was trying to say that if you find yourself spending a lot of time struggling with the script, definitely down be afraid to strip it down, change how it works, or remove it completely.

FWIW, I consider the build.py script a failed experiment. The idea itself was not completely bad, but the lack of buy-in from sufficient amount of developers meant that it never really had a chance of fulfilling its (rather lofty) goals.

I think it should be removed or at least scaled back significantly. The only thing it's really used for these days is to invoke msvc and clang-cl, and I'm sure we can achieve the same thing with some lit substitutions as well...

The script actually worked fairly well for Windows on Arm use-case. Although some tests like this one is written with only x86/x64 windows in mind so there will be minor tweaking required here n there.

To put it in a more constructive light, what I was trying to say that if you find yourself spending a lot of time struggling with the script, definitely down be afraid to strip it down, change how it works, or remove it completely.

I also plan to do some windows cleanup but immediate goal is to have LLDB windows Arm buildbot up and running with maximum number of passing tests.

labath accepted this revision.Feb 9 2022, 3:38 AM
This revision is now accepted and ready to land.Feb 9 2022, 3:38 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptFeb 9 2022, 4:40 AM