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,RFC] CSE path following on basic blocks


On 11/28/06, Paolo Bonzini <paolo.bonzini@lu.unisi.ch> wrote:
Steven Bosscher wrote:
> On 11/28/06, Eric Botcazou <ebotcazou@libertysurf.fr> wrote:
>> !   /* There is a previous path.  Make sure it started with FIRST_BB.  */
>> !   if (path_size)
>> !     gcc_assert (data->path[0].bb = first_bb);
>>
>> Not sure what you're trying to do here.  Or maybe an annoying typo?
>
> If there is a path in data, then we're going to discover a new path
> starting with
> data->path[0].bb.  But Passing first_bb may be unnecessary.

But you have only one '=' above... :-)

Ah. How silly...


That could also explain why I get different code with checking enabled
vs. checking disabled ;-)


Then you could also replace "label:" with "label:;"

And so it is done. I didn't know you could do this. Thanks!


Gr.
Steven


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