This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Make sure we use POSIX paths in header_information.py
ClosedPublic

Authored by ldionne on Jun 1 2023, 11:20 AM.

Details

Summary

Otherwise, the various lists of headers have different content based
on whether they are run on Windows or other platforms, which makes it
really difficult to write .gen.py tests correctly.

Diff Detail

Event Timeline

ldionne created this revision.Jun 1 2023, 11:20 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 1 2023, 11:20 AM
Herald added a subscriber: arichardson. · View Herald Transcript
ldionne requested review of this revision.Jun 1 2023, 11:20 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 1 2023, 11:20 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript

LGTM!

libcxx/utils/libcxx/test/header_information.py
133–136

I really dislike this nested os.path.dirname, but it is pre-existing.

Mordante accepted this revision.Jun 1 2023, 12:56 PM

Now with approval selected ;-)

This revision is now accepted and ready to land.Jun 1 2023, 12:56 PM