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] Fix part of pr25505

Daniel Berlin wrote:
>>
>> This code was all there before -- I just added code above it to detect
>> and handle the "simple decl" case.  It shows up in the diff because of
>> extra indentation.
>>
> I believe you :)
> But it's still going to give you strictly worse results for no good reason.

OK, I'm totally confused.  The logic you objected to is exactly what is
in mainline gcc today.  I proposed to augment this logic with:

if (VAR_DECL)
  check decl and subdecls for call-clobber
else
  { existing logic here }

How is that going to give strictly worse results?

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