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] Builtins handling in IVOPT


Hi,

> This patch works on the intrinsic calls handling issue in IVOPT mentioned here:
> http://gcc.gnu.org/ml/gcc-patches/2010-10/msg01295.html
> 
> In find_interesting_uses_stmt, it changes
> 
> arg = expr
> __builtin_xxx (arg)
> 
> to
> 
> arg = expr;
> tmp = addr_expr (mem_ref(arg));
> __builtin_xxx (tmp, ...)

this looks a bit confusing (and wasteful) to me. It would make more sense to
just record the argument as USE_ADDRESS and do the rewriting in rewrite_use_address.

Zdenek


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