This is an archive of the discontinued LLVM Phabricator instance.

[MC][test] Reorganize macro tests
ClosedPublic

Authored by MaskRay on Apr 12 2020, 2:42 PM.

Details

Summary
  • Add locations
  • Use FileCheck --match-full-lines --strict-whitespace
  • Merge related tests
  • Delete redundant tests
  • Improve test coverage

Diff Detail

Event Timeline

MaskRay created this revision.Apr 12 2020, 2:42 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 12 2020, 2:42 PM
MaskRay updated this revision to Diff 256882.Apr 12 2020, 2:45 PM

Forgot to add macro-arg.s

MaskRay updated this revision to Diff 256884.Apr 12 2020, 2:52 PM

More tests

MaskRay updated this revision to Diff 256887.Apr 12 2020, 3:14 PM
MaskRay edited the summary of this revision. (Show Details)

Merge tests

This revision was not accepted when it landed; it landed in state Needs Review.Apr 12 2020, 3:29 PM
This revision was automatically updated to reflect the committed changes.
Harbormaster failed remote builds in B52874: Diff 256880!
Harbormaster failed remote builds in B52875: Diff 256882!
Harbormaster failed remote builds in B52876: Diff 256884!
llvm/test/MC/AsmParser/macros-off.s