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: [IFUNC] PATCH: Add an ifunc attribute


On Sun, Jun 28, 2009 at 9:17 AM, Richard
Guenther<richard.guenther@gmail.com> wrote:
> On Sun, Jun 28, 2009 at 5:10 PM, H.J. Lu<hongjiu.lu@intel.com> wrote:
>> Hi,
>>
>> Here is a patch to add ifunc attribute, including many testscases
>> in C and C++. ?Any comments?
>
> +/* Get the function return type inside function body. ?Return a pointer
> + ? to the function for IFUNC function. ?*/
> +
> +static inline tree
> +function_return_type (const_tree decl)
> +{
> + ?if (DECL_IS_IFUNC (decl))
> + ? ?return build_pointer_type (TREE_TYPE (decl));
> + ?else
> + ? ?return TREE_TYPE (TREE_TYPE (decl));
> +}
>
> that's just ... gross. ?Instead adjust the function signature properly.
> Which of course makes your proposal not viable at all.
>
> I think your solution is overdesigned. ?Do you really expect so much
> usage that you need to make it easy to use from C++ (easy as in
> not mangle manually)?
>

I was told that people do want to use ifunc attribute in C++ class.


-- 
H.J.


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