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: RFA: Enhance doloop_condition_get()

Hi Revital1,

This looks similar to the ARM doloop pattern.

Ah, sorry - I had missed that.

I've committed a patch to recognize it which was approved by
Zdenek. please see -
http://gcc.gnu.org/ml/gcc-patches/2011-01/msg02225.html.
Maybe your patch can be integrated with my patch to avoid future collisions
?

Actually I think that with your patch applied I could just use version 3) of the supported doloop patterns and not need my patch at all.

Since the doloop part of your patch is approved, could you commit that now ?

As it happens I am an ARM maintainer, so I approve the ARM part of your patch.

Now you just need someone to review the modulo-sched.c part of the patch. Am I correct in thinking that without this part of the patch you just get poorer scheduling rather than broken code ?

Cheers
  Nick


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