On Thu, Jun 2, 2011 at 08:46, Steven Bosscher <stevenb....@gmail.com> wrote: > On Thu, Jun 2, 2011 at 1:46 PM, Diego Novillo <dnovi...@google.com> wrote: >> On Wed, Jun 1, 2011 at 21:03, Sharad Singhai <sing...@google.com> wrote: >> >>> 2011-06-01 Sharad Singhai <sing...@google.com> >>> >>> Google Ref 39994 >>> * ifcvt.c (noce_try_cmove_arith): Use the locus information >>> from the if-statment rather than the then path. >> >> Could you elaborate how it improves locus information? Is there a >> test case you can add to the testsuite? Or an example code fragment >> that shows how is the locus better now? >> >> OK for google/main. > > Why can't this patch just go onto the trunk?
Yes. Every patch submitted for google/main also applies to trunk. I generally try to avoid approving patches that are not inside my maintenance areas for trunk (unless the patch is obvious). Sharad simply forgot to request trunk approval for this patch and I forgot to remind him. Sharad, could you mark future patches? > (Idem for some other google/main patches -- is there a merge plan??) The merge plan is to submit patches to trunk. They are quickly moved in google/main for scheduling reasons, but everything going to google/main is automatically assumed to apply to trunk as well. The only patches that we don't necessarily mean to apply to trunk are the ones we put in google/integration (though some patches have already been moved or approved for trunk). Diego.
