Re: [Commited] Fix for PR59029

2013-11-07 Thread Yury Gribov
>> Should I fix the changelog with another commit? > Yes, just fix up the ChangeLog and commit Fixed. -Y

Re: [Commited] Fix for PR59029

2013-11-07 Thread Jakub Jelinek
On Thu, Nov 07, 2013 at 04:48:10PM +0400, Yury Gribov wrote: > Yup, Jakub already pointed this out. Should I fix the changelog with > another commit? Yes, just fix up the ChangeLog and commit, for ChangeLog changes of course you don't add a new ChangeLog entry. Jakub

Re: [Commited] Fix for PR59029

2013-11-07 Thread Yury Gribov
Garbuzov , Evgeny Gavrin Subject: Re: [Commited] Fix for PR59029 On 11/07/2013 04:38 PM, Marek Polacek wrote: On Thu, Nov 07, 2013 at 04:32:56PM +0400, Yury Gribov wrote: + PR sanitizer/59029 + * gcc/asan.c (get_mem_refs_of_builtin_call): Allow + integer literals as addresses in

Re: [Commited] Fix for PR59029

2013-11-07 Thread Marek Polacek
On Thu, Nov 07, 2013 at 04:32:56PM +0400, Yury Gribov wrote: > + PR sanitizer/59029 > + * gcc/asan.c (get_mem_refs_of_builtin_call): Allow > + integer literals as addresses in instrumented builtins. > + The prefix gcc/ hasn't been dropped as it should. Marek

[Commited] Fix for PR59029

2013-11-07 Thread Yury Gribov
Preapproved by Jakub in http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59029 diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 4991a3a..535b670 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2013-11-07 Yury Gribov + Jakub Jelinek + + PR sanitizer/59029 + * gcc/asan.c (get_me