This is an archive of the discontinued LLVM Phabricator instance.

Avoid u8"" literals in tests, their type changes in C++20
ClosedPublic

Authored by massberg on Jan 10 2023, 3:39 AM.

Details

Summary

Just specify the encoded bytes instead, which causes less confusion anyway.

Diff Detail

Event Timeline

massberg created this revision.Jan 10 2023, 3:39 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 10 2023, 3:39 AM
massberg requested review of this revision.Jan 10 2023, 3:39 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 10 2023, 3:39 AM
ilya-biryukov accepted this revision.Jan 10 2023, 4:35 AM

LGTM. The presubmit failures are definitely unrelated as they are in a lit test for a different tool.

This revision is now accepted and ready to land.Jan 10 2023, 4:35 AM
This revision was landed with ongoing or failed builds.Jan 10 2023, 6:15 AM
This revision was automatically updated to reflect the committed changes.