This is an archive of the discontinued LLVM Phabricator instance.

[Tooling/Includsion] Add the missing NULL symbol to the table.
ClosedPublic

Authored by hokein on Feb 24 2023, 2:48 AM.

Diff Detail

Event Timeline

hokein created this revision.Feb 24 2023, 2:48 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 24 2023, 2:48 AM
hokein requested review of this revision.Feb 24 2023, 2:48 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 24 2023, 2:48 AM
kadircet added inline comments.Feb 24 2023, 2:52 AM
clang/lib/Tooling/Inclusions/Stdlib/StdSpecialSymbolMap.inc
242

we use None for global namespace rather than leaving empty. let's keep it that way?

hokein updated this revision to Diff 500126.Feb 24 2023, 3:12 AM

oops, fix the patch

kadircet accepted this revision.Feb 24 2023, 3:30 AM

thanks!

This revision is now accepted and ready to land.Feb 24 2023, 3:30 AM
This revision was landed with ongoing or failed builds.Feb 24 2023, 4:00 AM
This revision was automatically updated to reflect the committed changes.
hokein marked an inline comment as done.