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 Thursday, March 13 2008, "Jakub Jelinek" wrote to "Jonathan Lennox, Uros Bizjak, Richard Guenther, gcc patches" saying:

> 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.

Ah, I see.  That makes sense.

> > > And I don't think we care or should support taking addresses of _mm_*
> > > functions, nobody says they are functions, that's an implementation detail.
> > > In fact some of them are even sometimes defined as macros, and you can't
> > > take address of a macro.
> > 
> > Paolo's suggestion of forbidding taking the address of __artificial__
> > functions seems like a good one.
> 
> Nope.

Right, I hadn't yet seen your earlier mail when I wrote that.

-- 
Jonathan Lennox
lennox at cs dot columbia dot edu


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