This is an archive of the discontinued LLVM Phabricator instance.

[Driver] Remove references to Solaris 12
ClosedPublic

Authored by ro on Aug 2 2023, 12:46 AM.

Details

Summary

Solaris 12 was re-christened as Solaris 11.4 while still in beta, so all references are long obsolete and can be removed.

Tested on amd64-pc-solaris2.11.

Diff Detail

Event Timeline

ro created this revision.Aug 2 2023, 12:46 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 2 2023, 12:46 AM
ro requested review of this revision.Aug 2 2023, 12:46 AM
MaskRay accepted this revision.Aug 2 2023, 1:15 PM

While you are modifying the lines static const char ...[] should look better than static const char *const SolarisX86Triples[]

This revision is now accepted and ready to land.Aug 2 2023, 1:15 PM
ro added a comment.Aug 3 2023, 12:51 AM

While you are modifying the lines static const char ...[] should look better than static const char *const SolarisX86Triples[]

While this would certainly avoid the ugly formatting, it doesn't compile. Besides, I'd rather avoid using different styles just a few lines apart.

This revision was automatically updated to reflect the committed changes.