This is an archive of the discontinued LLVM Phabricator instance.

[llvm-exegesis] Remove Assembler Tests
ClosedPublic

Authored by aidengrossman on May 3 2023, 11:45 PM.

Details

Summary

The Assembler tests have been disabled for years in tree and at this
point don't test anything other than common MC infrastructure that is
already tested in other parts of the tree. This patch removes them due
to the mentioned reasons.

Diff Detail

Event Timeline

aidengrossman created this revision.May 3 2023, 11:45 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 3 2023, 11:45 PM
Herald added a subscriber: mstojanovic. · View Herald Transcript
aidengrossman requested review of this revision.May 3 2023, 11:45 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 3 2023, 11:45 PM

I might just be missing something here, but the only thing these unittest files seem to be testing is common MC infrastructure. On top of that they've been disabled for an incredibly long time as mentioned in the commit message.

courbet accepted this revision.May 10 2023, 12:24 AM
This revision is now accepted and ready to land.May 10 2023, 12:24 AM
This revision was automatically updated to reflect the committed changes.
llvm/unittests/tools/llvm-exegesis/ARM/CMakeLists.txt