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.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
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.
Comment Actions
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.