This is an archive of the discontinued LLVM Phabricator instance.

[clang] Switch arm-mve-intrinsics tests to use %clang_cc1.
ClosedPublic

Authored by simon_tatham on Oct 25 2019, 3:26 AM.

Details

Summary

It isn't really necessary for them to run the clang driver, and it's
more efficient not to (and also more stable against driver changes).
Now they invoke cc1 directly, more like the analogous NEON tests.

Diff Detail

Event Timeline

simon_tatham created this revision.Oct 25 2019, 3:26 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 25 2019, 3:26 AM
simon_tatham edited the summary of this revision. (Show Details)Oct 25 2019, 3:27 AM
dmgreen accepted this revision.Oct 25 2019, 3:37 AM

Sounds good.

But make sure that -fno-discard-value-names is still included. Otherwise the names like "entry" will not be output, so the tests fail in "non-assert" builds.

This revision is now accepted and ready to land.Oct 25 2019, 3:37 AM

There's no need, as far as I can see. -fno-discard-value-names is a driver option only. On the cc1 command line the option is -discard-value-names, and it has no negative form: you turn it off by not specifying it in the first place, which indeed these command lines don't.

Ah, I see, that's why it was needed. That makes sense.

This revision was automatically updated to reflect the committed changes.