This is an archive of the discontinued LLVM Phabricator instance.

[libc++][NFC] Rename __libcpp_*_or_builtin to __constexpr_*
ClosedPublic

Authored by philnik on Dec 7 2022, 3:07 AM.

Details

Summary

We normally use the __libcpp_ prefix to backport functions, but that doesn't really make sense here. The functions are always available. Using __constexpr_ makes the intended use of the functions clear.

Diff Detail

Event Timeline

philnik created this revision.Dec 7 2022, 3:07 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 7 2022, 3:07 AM
philnik requested review of this revision.Dec 7 2022, 3:07 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 7 2022, 3:07 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript

Typically our internal functions like these are name __libcpp_foo, not __libcpp_foo_or_builtin. I see you now rename them to __constexpr_foo. Can you add comment to the commit message why you decided to use this new naming scheme?

I see the CI is red, but it just seems an internal test failure.

philnik edited the summary of this revision. (Show Details)Dec 7 2022, 9:30 AM
philnik updated this revision to Diff 480957.Dec 7 2022, 9:35 AM

Try to fix CI

Mordante accepted this revision.Dec 7 2022, 11:08 AM

Thanks for the update! LGTM when the CI is happy.

This revision is now accepted and ready to land.Dec 7 2022, 11:08 AM
This revision was landed with ongoing or failed builds.Dec 8 2022, 12:36 AM
This revision was automatically updated to reflect the committed changes.