This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Extend check for non-ASCII characters to src/, test/ and benchmarks/
ClosedPublic

Authored by ldionne on Aug 18 2022, 2:50 PM.

Diff Detail

Event Timeline

ldionne created this revision.Aug 18 2022, 2:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 18 2022, 2:50 PM
Herald added a subscriber: arichardson. · View Herald Transcript
ldionne requested review of this revision.Aug 18 2022, 2:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 18 2022, 2:50 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Mordante accepted this revision.Aug 20 2022, 7:22 AM
Mordante added a subscriber: Mordante.

Thanks for working on this! LGTM!

libcxx/utils/ci/run-buildbot
193

I like this exclusion list!

This revision is now accepted and ready to land.Aug 20 2022, 7:22 AM

The CI failure is a timeout. Shipping this.

This revision was landed with ongoing or failed builds.Aug 23 2022, 3:37 PM
This revision was automatically updated to reflect the committed changes.