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 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 don't.

Richard.


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