Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Traits.h: Fix feature-test macro for P1144 trivial relocatability #2314

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Quuxplusone
Copy link
Contributor

Well, this is awkward. #2216 added this #if check, but with the P1144 macro typo'ed: __cpp_lib_is_trivially_relocatable should have been __cpp_lib_trivially_relocatable.

In #2216 I even pointed to both the proposal and another repo's (correct) example, and yet somehow I still typo'ed it in the actual PR, and didn't notice for months! Really sorry about this.

Compiler Explorer (left-hand pane: as it is today; right-hand pane: with the wrong macro manually defined so that you get the codegen you were supposed to be getting all along)
https://godbolt.org/z/7oP3Yz61d

Attn: @Orvid, @ot

Well, this is awkward. facebook#2216 added this `#if` check, but with the
wrong macro: `__cpp_lib_is_trivially_relocatable` should have been
`__cpp_lib_trivially_relocatable`.
@facebook-github-bot
Copy link
Contributor

@Orvid has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants