Re: [PING^2][PATCH] Fix documentation of -mpoke-function-name ARM option

2020-03-23 Thread Richard Earnshaw
On 22/03/2020 18:15, Jérémy Lefaure wrote: > Hi Wilco, > > On Mon, Mar 09, 2020 at 05:53:41PM +, Wilco Dijkstra wrote: >> Hi, >> >> There is no single PC offset that is correct given CPUs may use different >> offsets. > > Isn't it always an offset of 8 in ARM mode and 4 bytes in Thumb mode

Re: [PING^2][PATCH] Fix documentation of -mpoke-function-name ARM option

2020-03-22 Thread Jérémy Lefaure
Hi Wilco, On Mon, Mar 09, 2020 at 05:53:41PM +, Wilco Dijkstra wrote: > Hi, > > There is no single PC offset that is correct given CPUs may use different > offsets. Isn't it always an offset of 8 in ARM mode and 4 bytes in Thumb mode ? At least in ARMv7 and in AArch32 state in ARMv8 ? >

Re: [PING^2][PATCH] Fix documentation of -mpoke-function-name ARM option

2020-03-09 Thread Wilco Dijkstra
Hi, There is no single PC offset that is correct given CPUs may use different offsets. GCC may also schedule the instruction that stores the PC. This feature used to work on early Arms but is no longer functional or useful today, so the best way forward is to remove it altogether. There are many

Re: [PING^2][PATCH] Fix documentation of -mpoke-function-name ARM option

2020-03-08 Thread blatinox
Hi Sandra, On Mon, Feb 24, 2020 at 02:19:21PM -0700, Sandra Loosemore wrote: > On 2/20/20 3:08 PM, Jérémy Lefaure wrote: > > Hello, > > > > Ping for https://gcc.gnu.org/ml/gcc-patches/2019-12/msg01081.html. > > > > Thank you, > > Jérémy > > > > On Sun, Dec 15, 2019 at 07:20:26PM +0100, Jérémy

Re: [PING^2][PATCH] Fix documentation of -mpoke-function-name ARM option

2020-02-24 Thread Sandra Loosemore
On 2/20/20 3:08 PM, Jérémy Lefaure wrote: Hello, Ping for https://gcc.gnu.org/ml/gcc-patches/2019-12/msg01081.html. Thank you, Jérémy On Sun, Dec 15, 2019 at 07:20:26PM +0100, Jérémy Lefaure wrote: Hi! Since in ARM state the value of PC is the address of the current instruction plus 8

[PING^2][PATCH] Fix documentation of -mpoke-function-name ARM option

2020-02-20 Thread Jérémy Lefaure
Hello, Ping for https://gcc.gnu.org/ml/gcc-patches/2019-12/msg01081.html. Thank you, Jérémy On Sun, Dec 15, 2019 at 07:20:26PM +0100, Jérémy Lefaure wrote: > Hi! > > Since in ARM state the value of PC is the address of the current > instruction plus 8 bytes, the code inspecting the value of PC