Re: [AArch64, Patch] Add range-check for Symbol + offset addressing.

2014-11-20 Thread Marcus Shawcroft
On 20 November 2014 14:33, Tejas Belagod wrote: > The same patch applies cleanly to 4.9. OK to commit? > > Thanks, > Tejas. Provided it regresses ok, yes. /Marcus

Re: [AArch64, Patch] Add range-check for Symbol + offset addressing.

2014-11-20 Thread Tejas Belagod
On 14/11/14 17:33, Marcus Shawcroft wrote: On 14 November 2014 08:12, Tejas Belagod wrote: 2014-11-14 Tejas Belagod gcc/ * config/aarch64/aarch64-protos.h (aarch64_classify_symbol): Fixup prototype. * config/aarch64/aarch64.c (aarch64_expand_mov_immediate,

Re: [AArch64, Patch] Add range-check for Symbol + offset addressing.

2014-11-14 Thread Marcus Shawcroft
On 14 November 2014 08:12, Tejas Belagod wrote: > 2014-11-14 Tejas Belagod > > gcc/ > * config/aarch64/aarch64-protos.h (aarch64_classify_symbol): > Fixup prototype. > * config/aarch64/aarch64.c (aarch64_expand_mov_immediate, > aarch64_cannot_force_const_mem, aa

Re: [AArch64, Patch] Add range-check for Symbol + offset addressing.

2014-11-14 Thread Andrew Pinski
On Fri, Nov 14, 2014 at 12:50 AM, Tejas Belagod wrote: > On 14/11/14 08:19, Andrew Pinski wrote: >> >> On Fri, Nov 14, 2014 at 12:12 AM, Tejas Belagod >> wrote: >>> >>> >>> Hi, >>> >>> Following the discussion here >>> https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02237.html, this has been >>> tr

Re: [AArch64, Patch] Add range-check for Symbol + offset addressing.

2014-11-14 Thread pinskia
> On Nov 14, 2014, at 12:54 AM, Marcus Shawcroft > wrote: > >> On 14 November 2014 08:19, Andrew Pinski wrote: >>> On Fri, Nov 14, 2014 at 12:12 AM, Tejas Belagod >>> wrote: >>> >>> Hi, >>> >>> Following the discussion here >>> https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02237.html, t

Re: [AArch64, Patch] Add range-check for Symbol + offset addressing.

2014-11-14 Thread Marcus Shawcroft
On 14 November 2014 08:19, Andrew Pinski wrote: > On Fri, Nov 14, 2014 at 12:12 AM, Tejas Belagod wrote: >> >> Hi, >> >> Following the discussion here >> https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02237.html, this has been >> tracked down to a range-checking bug with symbol + offset style addr

Re: [AArch64, Patch] Add range-check for Symbol + offset addressing.

2014-11-14 Thread Tejas Belagod
On 14/11/14 08:19, Andrew Pinski wrote: On Fri, Nov 14, 2014 at 12:12 AM, Tejas Belagod wrote: Hi, Following the discussion here https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02237.html, this has been tracked down to a range-checking bug with symbol + offset style addressing with adrp where w

Re: [AArch64, Patch] Add range-check for Symbol + offset addressing.

2014-11-14 Thread Andrew Pinski
On Fri, Nov 14, 2014 at 12:12 AM, Tejas Belagod wrote: > > Hi, > > Following the discussion here > https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02237.html, this has been > tracked down to a range-checking bug with symbol + offset style addressing > with adrp where we allowed any arbitrary offset