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, i386]: Fix PR target/35553, -fkeep-inline-functions and -O errors our in SSE headers


On Thu, Mar 13, 2008 at 03:24:26PM +0100, Richard Guenther wrote:
> On Thu, Mar 13, 2008 at 3:12 PM, Jakub Jelinek <jakub@redhat.com> wrote:
> >  Compile that with -O0 -fdump-tree-apply_inline and also with -O1 -fdump-tree-optimized.
> >  You'll see that in apply_inline dump which is the last before expansion you
> >  have
> >   i = 6;
> >   baz (i);
> >  and therefore the builtin expansion won't see that the argument is
> >  INTEGER_CST.  In the optimized dump there is baz (6);, so the expander sees
> >  it.
> 
> That could be fixed if we were going into-SSA for -O0 via the constant
> propagation done at inlining time.  Then you'd get the same behavior
> as for the macro.

But such a change would make the code less debuggable.  When stepping
into the inline function, you couldn't see what value does the i argument
have, nor you could change it before calling the other function.

	Jakub


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