[PATCH] Refactor duplicate functions

Bernie Ogden bogden at arm.com
Thu Dec 12 05:35:42 PST 2013


Thanks both - original change + refactoring commited as r197151-4.

Tim, no worries about the delay. I could've driven it harder, and the
discusson about targets/driver design was educational.

Regards,

Bernie

> -----Original Message-----
> From: Jim Grosbach [mailto:grosbach at apple.com]
> Sent: 12 December 2013 02:45
> To: Bernard Ogden
> Cc: Renato Golin; reviews+D2243+public+6ca39a6218cf2957 at llvm-
> reviews.chandlerc.com; cfe-commits at cs.uiuc.edu; Tim Northover
> Subject: Re: [PATCH] Refactor duplicate functions
> 
> Since Tim's OK with it, that's good enough for me. If there are any
> problems, we can sort it out post-commit.
> 
> Thanks!
> -Jim
> 
> On Dec 11, 2013, at 5:54 AM, Tim Northover <t.p.northover at gmail.com>
> wrote:
> 
> >> I think the patch is good, if not pretty, so I'd like to commit. Can
> anyone
> >> comment on whether the tests are enough to guarantee preservation of
> the
> >> existing Darwin behaviour?
> >
> > I think they looks reasonable. They cover the things I care about at
> > the moment anyway. I think you should go for it. (And very sorry for
> > the long delay in getting this committed. I know it wasn't exactly
> > your idea in the first place).
> >
> > Cheers.
> >
> > Tim.
> 








More information about the cfe-commits mailing list