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] Attribute noclone


On 07/17/2009 12:30 PM, Martin Jambor wrote:
Hi,

since the  attribute noinline does not prevent  ipa-cp cloning from
taking place, there  has been a need to provide  a new attribute to
do just that.

What, specifically, is this need?


The way I see it is that your vision in this is too narrow,
and you're not considering other reasons that a function
might be cloned.

If we add a noclone attribute, and it prohibits all clones,
then users that think they're working around bugs/misoptimizations
in ipa-cp may wind up unintentionally prohibiting transactional
memory clones as well.

If we add a noclone attribute that merely disables ipa-cp
(which is what you've done), then the attribute is mis-named
because some clones are in fact allowed.

So what real problem is it that you're trying to solve?


r~



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