[PATCH] Add attribute "artificial"
Mark Mitchell
mark@codesourcery.com
Fri Aug 5 23:00:00 GMT 2005
Mark Mitchell wrote:
> Devang Patel wrote:
>
>> As we discussed on Friday, this patch adds new attribute - artificial.
>>
>> STABS:
>>
>> We do not need to set DECL_IGNORED_P bit. So, instead of removing
>> debug info. this patch updates the debug info. Inliner is updated to
>> adjust location info when "artificial" attribute is seen.
>>
>> DWARF:
>>
>> Setting DECL_ARTIFICIAL did not work because it does not survive
>> after duplicate_decls() etc. Plus C++ version takes different action
>> based on whether DECL_ARTIFICIAL is set or not. If we really want to
>> pursue that route then further changes will be required. This patch
>> does not overload DECL_ARTIFICIAL and keeps everything clean and
>> separate.
>
>
> You are correct: setting DECL_ARTIFICIAL is not right. DECL_IGNORED+P
> is what we should set. And, that should be debug backend independent;
> there should be no need to modify the DWARF or DBX backends.
OK, Dan tells me that bringing up DECL_IGNORED_P was idiotic. So, I'll
retract that bit. I do still think that "artificial" is a weird name
for this attribute. It doesn't mean anything to people who aren't GCC
hackers.
--
Mark Mitchell
CodeSourcery, LLC
mark@codesourcery.com
(916) 791-8304
More information about the Gcc-patches
mailing list