[patch c++]: PR/47213] New: ICE: SIGSEGV in determine_visibility (decl2.c:2076) with -fvisibility-ms-compat
Jason Merrill
jason@redhat.com
Wed Jan 12 18:59:00 GMT 2011
On 01/12/2011 04:07 AM, Kai Tietz wrote:
> - warning (OPT_Wattributes, "visibility attribute not supported "
> - "in this configuration; ignored");
> + if (decl && lookup_attribute ("dllexport", DECL_ATTRIBUTES (decl)))
> + warning (OPT_Wattributes, "visibility attribute not supported "
> + "in this configuration; ignored");
This change is wrong. If we don't support visibility on a target,
trying to apply it should give a warning by default. It certainly
doesn't make any sense to condition that warning on the dllexport
attribute in target-independent code.
Perhaps we want a target-specific assemble_visibility hook for PE, or to
just ignore -fvisibility-ms-compat.
Jason
More information about the Gcc-patches
mailing list