This is an archive of the discontinued LLVM Phabricator instance.

[clang][safestack] Remove triple from stack usage test
ClosedPublic

Authored by paulkirth on Apr 21 2022, 2:51 PM.

Details

Summary

Supplying the target triple caused breakeages for compilers that don't support
the supplied triple.

Diff Detail

Event Timeline

paulkirth created this revision.Apr 21 2022, 2:51 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 21 2022, 2:51 PM
paulkirth requested review of this revision.Apr 21 2022, 2:51 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 21 2022, 2:51 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
mysterymath accepted this revision.Apr 21 2022, 2:56 PM
This revision is now accepted and ready to land.Apr 21 2022, 2:56 PM
This revision was landed with ongoing or failed builds.Apr 21 2022, 2:58 PM
This revision was automatically updated to reflect the committed changes.

Thank you @paulkirth. I have tested these changes locally on the builder. It works fine.

unfortunately it breaks many other tests, so we probably need to revert this