This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch, ARM, v2] Enable libsanitizer
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Christophe Lyon <christophe dot lyon at linaro dot org>
- Cc: Konstantin Serebryany <konstantin dot s dot serebryany at gmail dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Patch Tracking <patches at linaro dot org>
- Date: Tue, 7 May 2013 14:32:42 +0200
- Subject: Re: [Patch, ARM, v2] Enable libsanitizer
- References: <CAKdteOY1Qoe1dXTSGSgABp3RPi+4+ge-XreNr9URK4kLjbfRQQ at mail dot gmail dot com> <20130507084430 dot GU28963 at tucnak dot redhat dot com> <CAKdteObuHp52gP=k5S-a7Dn22Yrb7awxizHbsGGkDbJ-UyPc3A at mail dot gmail dot com> <20130507122223 dot GB28963 at tucnak dot redhat dot com> <CAKdteObsARrk6JfzeC8z-p8NxWD9KDOwLUTfcov6DcNvuTWfEA at mail dot gmail dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Tue, May 07, 2013 at 02:31:05PM +0200, Christophe Lyon wrote:
> On 7 May 2013 14:22, Jakub Jelinek <jakub@redhat.com> wrote:
> > On Tue, May 07, 2013 at 02:18:35PM +0200, Christophe Lyon wrote:
> >> Thanks for your review, here is a new version taking your comments into account.
> >>
> >> Christophe.
> >>
> >> 2013-05-06 Christophe Lyon <christophe.lyon@linaro.org>
> >
> > Two spaces before <, instead of just one.
> > Otherwise it looks good to me, but if it depends on the libsanitizer/ change
> > now excluded from the patch, has that been added upstream?
> No, it was a mistake of mine.
>
> So, it's OK to commit (with the ChangeLog fix)?
Yes, thanks.
Jakub
- References:
- [Patch, ARM, v2] Enable libsanitizer
- Re: [Patch, ARM, v2] Enable libsanitizer
- Re: [Patch, ARM, v2] Enable libsanitizer
- Re: [Patch, ARM, v2] Enable libsanitizer
- Re: [Patch, ARM, v2] Enable libsanitizer