diff options
author | Ian Lance Taylor <iant@golang.org> | 2020-07-10 13:43:09 -0700 |
---|---|---|
committer | Ian Lance Taylor <iant@golang.org> | 2020-07-11 12:41:28 -0700 |
commit | e109f6e438b72ef3e403162971068d28d09b82f5 (patch) | |
tree | 8ead189e618f8ef1456c8b02c81de0cc1585d8a6 /gcc/go/gofrontend/names.cc | |
parent | e7d6d8f9f7b687c8e7438523d7d6f7a539ec7287 (diff) | |
download | gcc-e109f6e438b72ef3e403162971068d28d09b82f5.zip gcc-e109f6e438b72ef3e403162971068d28d09b82f5.tar.gz gcc-e109f6e438b72ef3e403162971068d28d09b82f5.tar.bz2 |
compiler: avoid generating unnamed bool type descriptor
We were generating it in cases where a boolean expression was
converted directly to an empty interface type.
Fixes golang/go#40152
Reviewed-on: https://go-review.googlesource.com/c/gofrontend/+/242002
Diffstat (limited to 'gcc/go/gofrontend/names.cc')
-rw-r--r-- | gcc/go/gofrontend/names.cc | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/gcc/go/gofrontend/names.cc b/gcc/go/gofrontend/names.cc index a721a36..1f0a545 100644 --- a/gcc/go/gofrontend/names.cc +++ b/gcc/go/gofrontend/names.cc @@ -975,7 +975,14 @@ Gogo::type_descriptor_name(const Type* type, Named_type* nt) return "unsafe.Pointer..d"; if (nt == NULL) - return "type.." + type->mangled_name(this); + { + // Sanity check: we should never generate a type descriptor for + // an unnamed primitive type. For those we should always be + // using a named type, like "int". + go_assert(!type->is_basic_type()); + + return "type.." + type->mangled_name(this); + } std::string ret; Named_object* no = nt->named_object(); |