Message ID | 20200807113221.2522574-1-ibuclaw@gdcproject.org |
---|---|
State | New |
Headers | show |
Series | libiberty: Add support for 'in' and 'in ref' storage classes. | expand |
Ping. Though I will hold back on supporting 'in ref' until it has been formalized. Current discussions are around what value might it have over 'ref const scope' (so far, none). Iain. On 07/08/2020 13:32, Iain Buclaw wrote: > Hi, > > This patch adds support for 'in' as a first-class storage class with its > own mangle symbol, of which also permits 'in ref'. Previously, 'in' was > an alias to 'const [scope]', which is a type constructor. > > The mangle symbol repurposed for this is 'I', which was originally used > by identifier types. However, while TypeIdentifier is part of the > grammar, it must be resolved to some other entity during the semantic > passes, and so shouldn't appear anywhere in the mangled name. > > OK for mainline? > > Regards > Iain. > > --- > libiberty/ChangeLog: > > * d-demangle.c (dlang_function_args): Handle 'in' and 'in ref' > parameter storage classes. > (dlang_type): Remove identifier type. > * testsuite/d-demangle-expected: Update tests. > --- > libiberty/d-demangle.c | 10 +++++++++- > libiberty/testsuite/d-demangle-expected | 16 ++++++++-------- > 2 files changed, 17 insertions(+), 9 deletions(-) > > diff --git a/libiberty/d-demangle.c b/libiberty/d-demangle.c > index f2d6946ecad..2e52eff7617 100644 > --- a/libiberty/d-demangle.c > +++ b/libiberty/d-demangle.c > @@ -699,6 +699,15 @@ dlang_function_args (string *decl, const char *mangled, struct dlang_info *info) > > switch (*mangled) > { > + case 'I': /* in(T) */ > + mangled++; > + string_append (decl, "in "); > + if (*mangled == 'K') /* in ref(T) */ > + { > + mangled++; > + string_append (decl, "ref "); > + } > + break; > case 'J': /* out(T) */ > mangled++; > string_append (decl, "out "); > @@ -826,7 +835,6 @@ dlang_type (string *decl, const char *mangled, struct dlang_info *info) > mangled = dlang_function_type (decl, mangled, info); > string_append (decl, "function"); > return mangled; > - case 'I': /* ident T */ > case 'C': /* class T */ > case 'S': /* struct T */ > case 'E': /* enum T */ > diff --git a/libiberty/testsuite/d-demangle-expected b/libiberty/testsuite/d-demangle-expected > index e3f32e31d7b..ffb317f23d4 100644 > --- a/libiberty/testsuite/d-demangle-expected > +++ b/libiberty/testsuite/d-demangle-expected > @@ -274,14 +274,6 @@ _D8demangle4testFNhG4dZv > demangle.test(__vector(double[4])) > # > --format=dlang > -_D8demangle4testFI5identZv > -demangle.test(ident) > -# > ---format=dlang > -_D8demangle4testFI5ident4testZv > -demangle.test(ident.test) > -# > ---format=dlang > _D8demangle4testFC5classZv > demangle.test(class) > # > @@ -314,6 +306,14 @@ _D8demangle4testFT7typedef4testZv > demangle.test(typedef.test) > # > --format=dlang > +_D8demangle4testFIaZv > +demangle.test(in char) > +# > +--format=dlang > +_D8demangle4testFIKaZv > +demangle.test(in ref char) > +# > +--format=dlang > _D8demangle4testFJaZv > demangle.test(out char) > # >
On Fri, 2020-08-07 at 13:32 +0200, Iain Buclaw via Gcc-patches wrote: > Hi, > > This patch adds support for 'in' as a first-class storage class with its > own mangle symbol, of which also permits 'in ref'. Previously, 'in' was > an alias to 'const [scope]', which is a type constructor. > > The mangle symbol repurposed for this is 'I', which was originally used > by identifier types. However, while TypeIdentifier is part of the > grammar, it must be resolved to some other entity during the semantic > passes, and so shouldn't appear anywhere in the mangled name. > > OK for mainline? > > Regards > Iain. > > --- > libiberty/ChangeLog: > > * d-demangle.c (dlang_function_args): Handle 'in' and 'in ref' > parameter storage classes. > (dlang_type): Remove identifier type. > * testsuite/d-demangle-expected: Update tests. OK jeff >
diff --git a/libiberty/d-demangle.c b/libiberty/d-demangle.c index f2d6946ecad..2e52eff7617 100644 --- a/libiberty/d-demangle.c +++ b/libiberty/d-demangle.c @@ -699,6 +699,15 @@ dlang_function_args (string *decl, const char *mangled, struct dlang_info *info) switch (*mangled) { + case 'I': /* in(T) */ + mangled++; + string_append (decl, "in "); + if (*mangled == 'K') /* in ref(T) */ + { + mangled++; + string_append (decl, "ref "); + } + break; case 'J': /* out(T) */ mangled++; string_append (decl, "out "); @@ -826,7 +835,6 @@ dlang_type (string *decl, const char *mangled, struct dlang_info *info) mangled = dlang_function_type (decl, mangled, info); string_append (decl, "function"); return mangled; - case 'I': /* ident T */ case 'C': /* class T */ case 'S': /* struct T */ case 'E': /* enum T */ diff --git a/libiberty/testsuite/d-demangle-expected b/libiberty/testsuite/d-demangle-expected index e3f32e31d7b..ffb317f23d4 100644 --- a/libiberty/testsuite/d-demangle-expected +++ b/libiberty/testsuite/d-demangle-expected @@ -274,14 +274,6 @@ _D8demangle4testFNhG4dZv demangle.test(__vector(double[4])) # --format=dlang -_D8demangle4testFI5identZv -demangle.test(ident) -# ---format=dlang -_D8demangle4testFI5ident4testZv -demangle.test(ident.test) -# ---format=dlang _D8demangle4testFC5classZv demangle.test(class) # @@ -314,6 +306,14 @@ _D8demangle4testFT7typedef4testZv demangle.test(typedef.test) # --format=dlang +_D8demangle4testFIaZv +demangle.test(in char) +# +--format=dlang +_D8demangle4testFIKaZv +demangle.test(in ref char) +# +--format=dlang _D8demangle4testFJaZv demangle.test(out char) #