On Wed, Mar 19, 2025 at 03:23:30PM +0100, Richard Biener wrote:
> The enum currently has a member named NONE which pollutes the global
> namespace unnecessarily.  Use a scoped enum instead.
> 
> Bootstrap and regtest running on x86_64-unknown-linux-gnu.
> 
> OK if that succeeds?
> 
> gcc/
>       * tree-core.h (function_decl_type): Make a scoped enum.
>       * tree.h (set_function_decl_type): Adjust.
>       (DECL_IS_OPERATOR_NEW_P): Likewise.
>       (DECL_SET_IS_OPERATOR_NEW): Likewise.
>       (DECL_IS_OPERATOR_DELETE_P): Likewise.
>       (DECL_SET_IS_OPERATOR_DELETE): Likewise.
>       (DECL_LAMBDA_FUNCTION_P): Likewise.
>       (DECL_SET_LAMBDA_FUNCTION): Likewise.
>       * lto-streamer-out.cc (hash_tree): Hash all of
>       FUNCTION_DECL_DECL_TYPE.
>       * tree-streamer-out.cc (pack_ts_function_decl_value_fields):
>       Adjust.
> 
> gcc/cp/
>       * module.cc (trees_out::core_bools): Convert scoped enum
>       explicitly.

LGTM

        Jakub

Reply via email to