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: [tree-ssa] Cleanup some redundant (?) code


On Thu, 09 Oct 2003 18:49:14 +0200, Steven Bosscher <s.bosscher@student.tudelft.nl> wrote:

> ;; Function __gcov_open (__gcov_open)
>  
> __gcov_open ()
> {
>
>   #   __gcov_var_2 = VDEF <__gcov_var_1>;
>   #   VUSE <__gcov_var_1>;
>   __gcov_var.offset = __gcov_var.length
> }
>
> Perhaps loads from union/struct members should also be gimplified into a
> load into a temporary...?

They are, except when used directly on the rhs of an assignment.

For a while I was working on a patch to require that an assignment have a
temporary on either the lhs or rhs, but it slowed things down quite a bit.
Perhaps only requiring that the lhs or rhs be a register would be better.

Jason


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