Bug 239482 - REGRESSION(r292863): std::remove_cvref_t not defined in GCC9<
Summary: REGRESSION(r292863): std::remove_cvref_t not defined in GCC9<
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Diego Pino
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-04-18 23:14 PDT by Diego Pino
Modified: 2022-04-27 00:24 PDT (History)
8 users (show)

See Also:


Attachments
Patch (1.53 KB, patch)
2022-04-18 23:18 PDT, Diego Pino
no flags Details | Formatted Diff | Diff
Patch (1.55 KB, patch)
2022-04-19 09:14 PDT, Diego Pino
no flags Details | Formatted Diff | Diff
Patch (1.55 KB, patch)
2022-04-20 19:39 PDT, Diego Pino
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Diego Pino 2022-04-18 23:14:56 PDT
REGRESSION(r292863): std::remove_cvref_t not defined in GCC9<
Comment 1 Diego Pino 2022-04-18 23:18:30 PDT
Created attachment 457858 [details]
Patch
Comment 2 Daniel Kolesa 2022-04-19 07:42:13 PDT
should probably make it conditional for `__GLIBCXX__` to avoid affecting libc++ environments (like mac)
Comment 3 Diego Pino 2022-04-19 09:14:43 PDT
Created attachment 457907 [details]
Patch
Comment 4 Zan Dobersek 2022-04-19 23:25:18 PDT
Comment on attachment 457907 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=457907&action=review

> Source/WTF/wtf/StdLibExtras.h:612
> +template <typename T>
> +struct remove_cvref {
> +    using type = typename std::remove_cv<typename std::remove_reference<T>::type>::type;
> +};

This has to be in the std namespace too.
Comment 5 Diego Pino 2022-04-20 19:39:32 PDT
Created attachment 458032 [details]
Patch
Comment 6 EWS 2022-04-21 01:34:48 PDT
zan@falconsigh.net does not have committer permissions according to https://raw.githubusercontent.com/WebKit/WebKit/main/metadata/contributors.json.

Rejecting attachment 458032 [details] from commit queue.
Comment 7 EWS 2022-04-21 01:38:31 PDT
zan@falconsigh.net does not have reviewer permissions according to https://raw.githubusercontent.com/WebKit/WebKit/main/metadata/contributors.json.

Rejecting attachment 458032 [details] from commit queue.
Comment 8 EWS 2022-04-21 02:30:07 PDT
Committed r293148 (249844@main): <https://commits.webkit.org/249844@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 458032 [details].
Comment 9 Ryan Haddad 2022-04-21 17:07:59 PDT
rdar://92082938