This is an archive of the discontinued LLVM Phabricator instance.

[OpaquePtrs][Clang] Add -no-opaque-pointers to tests
ClosedPublic

Authored by nikic on Apr 5 2022, 3:35 AM.

Details

Summary

This adds -no-opaque-pointers to clang tests whose output will change when opaque pointers are enabled by default. This is intended to be part of the migration approach described in https://discourse.llvm.org/t/enabling-opaque-pointers-by-default/61322/9.

The patch has been produced by replacing %clang_cc1 with %clang_cc1 -no-opaque-pointers for tests that fail with opaque pointers enabled. Worth noting that this doesn't cover all tests, there's a remaining 60 tests not using %clang_cc1 that will need a followup change (probably best to migrate them to %clang_cc1, as they don't look to be using the driver interface intentionally).

Diff Detail

Event Timeline

nikic created this revision.Apr 5 2022, 3:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 5 2022, 3:35 AM
nikic requested review of this revision.Apr 5 2022, 3:35 AM
aeubanks accepted this revision.Apr 6 2022, 1:22 PM
This revision is now accepted and ready to land.Apr 6 2022, 1:22 PM
This revision was landed with ongoing or failed builds.Apr 7 2022, 3:10 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptApr 7 2022, 3:11 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript