This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PATCH: __nodebug__ attribute for use on SSE intrinsic wrappers



On Jul 29, 2005, at 2:39 PM, Geoff Keating wrote:


On 29/07/2005, at 1:45 PM, Devang Patel wrote:


We're converging towards one solution (that is, support new attribute, decorate intrinsic headers, subtract in stabs and use DW_AT_ARTIFICIAL in DWARF). Now, once 1) one of the GWP OK it in principal and 2) we select name of new attribute, Fariborz (or I) can prepare updated patch.


I think this is OK in principle, and there seems to be agreement on this (somewhat violent agreement, but I'm pretty sure it's agreement). I'm not sure if DW_AT_artificial is an exact match, but the only alternative I see would be a GNU-specific attribute, and I think we should try DW_AT_artificial before going to such lengths.


Marking these functions as DECL_ARTIFICIAL has other consequences, but I don't *think* any of them are undesirable, so I suggest trying that as an implementation technique.

When you document the new attribute, I'd suggest saying that it should only be used in header files that are part of the compiler, or something like that.

OK. I will prepare a patch based on this.


-
Devang


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]