Message ID | 20231113223542.11562-1-cupertino.miranda@oracle.com |
---|---|
State | New |
Headers | show |
Series | bpf: Delayed the removal of the parser enum plugin handler. | expand |
On 11/13/23 14:35, Cupertino Miranda wrote: > The parser plugin handler that is responsible for collecting enum values > information was being removed way too early. > bpf_resolve_overloaded_core_builtin is called by the parser. > It was moved to the function execute_lower_bpf_core. > OK, thanks. > gcc/ChangeLog: > * config/bpf/core-builtins.cc > (bpf_resolve_overloaded_core_builtin): Removed call. > (execute_lower_bpf_core): Added all to remove_parser_plugin. > --- > gcc/config/bpf/core-builtins.cc | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/gcc/config/bpf/core-builtins.cc b/gcc/config/bpf/core-builtins.cc > index a224847d5d91..2ba78d7aed2e 100644 > --- a/gcc/config/bpf/core-builtins.cc > +++ b/gcc/config/bpf/core-builtins.cc > @@ -1473,8 +1473,6 @@ tree > bpf_resolve_overloaded_core_builtin (location_t loc, tree fndecl, > void *arglist) > { > - remove_parser_plugin (); > - > if (!bpf_require_core_support ()) > return error_mark_node; > > @@ -1688,6 +1686,7 @@ make_gimple_core_safe_access_index (tree *tp, > static unsigned int > execute_lower_bpf_core (void) > { > + remove_parser_plugin (); > if (!TARGET_BPF_CORE) > return 0; >
Thanks!! Commited ! David Faust writes: > On 11/13/23 14:35, Cupertino Miranda wrote: >> The parser plugin handler that is responsible for collecting enum values >> information was being removed way too early. >> bpf_resolve_overloaded_core_builtin is called by the parser. >> It was moved to the function execute_lower_bpf_core. >> > > OK, thanks. > >> gcc/ChangeLog: >> * config/bpf/core-builtins.cc >> (bpf_resolve_overloaded_core_builtin): Removed call. >> (execute_lower_bpf_core): Added all to remove_parser_plugin. >> --- >> gcc/config/bpf/core-builtins.cc | 3 +-- >> 1 file changed, 1 insertion(+), 2 deletions(-) >> >> diff --git a/gcc/config/bpf/core-builtins.cc b/gcc/config/bpf/core-builtins.cc >> index a224847d5d91..2ba78d7aed2e 100644 >> --- a/gcc/config/bpf/core-builtins.cc >> +++ b/gcc/config/bpf/core-builtins.cc >> @@ -1473,8 +1473,6 @@ tree >> bpf_resolve_overloaded_core_builtin (location_t loc, tree fndecl, >> void *arglist) >> { >> - remove_parser_plugin (); >> - >> if (!bpf_require_core_support ()) >> return error_mark_node; >> >> @@ -1688,6 +1686,7 @@ make_gimple_core_safe_access_index (tree *tp, >> static unsigned int >> execute_lower_bpf_core (void) >> { >> + remove_parser_plugin (); >> if (!TARGET_BPF_CORE) >> return 0; >>
diff --git a/gcc/config/bpf/core-builtins.cc b/gcc/config/bpf/core-builtins.cc index a224847d5d91..2ba78d7aed2e 100644 --- a/gcc/config/bpf/core-builtins.cc +++ b/gcc/config/bpf/core-builtins.cc @@ -1473,8 +1473,6 @@ tree bpf_resolve_overloaded_core_builtin (location_t loc, tree fndecl, void *arglist) { - remove_parser_plugin (); - if (!bpf_require_core_support ()) return error_mark_node; @@ -1688,6 +1686,7 @@ make_gimple_core_safe_access_index (tree *tp, static unsigned int execute_lower_bpf_core (void) { + remove_parser_plugin (); if (!TARGET_BPF_CORE) return 0;