Re: [RFC][PATCH] Remove SLOW_BYTE_ACCESS

2022-11-09 Thread Andrew Pinski via Gcc-patches
On Fri, Nov 17, 2017 at 11:08 AM Wilco Dijkstra wrote: > > Remove SLOW_BYTE_ACCESS given it's confusing, badly named, > badly documented and used incorrectly. Although most targets > define it as 1, there are several targets which confuse it > (based on comments next to it) and set it to 0 since

Re: [RFC][PATCH] Remove SLOW_BYTE_ACCESS

2017-11-17 Thread Jeff Law
On 11/17/2017 12:01 PM, Wilco Dijkstra wrote: > Remove SLOW_BYTE_ACCESS given it's confusing, badly named, > badly documented and used incorrectly. Although most targets > define it as 1, there are several targets which confuse it > (based on comments next to it) and set it to 0 since the name >

Re: [RFC][PATCH] Remove SLOW_BYTE_ACCESS

2017-11-17 Thread Palmer Dabbelt
On Fri, 17 Nov 2017 11:01:21 PST (-0800), wilco.dijks...@arm.com wrote: Remove SLOW_BYTE_ACCESS given it's confusing, badly named, badly documented and used incorrectly. Although most targets define it as 1, there are several targets which confuse it (based on comments next to it) and set it to

Re: [RFC][PATCH] Remove SLOW_BYTE_ACCESS

2017-11-17 Thread Joseph Myers
If removing a target macro, it should be poisoned in system.h. -- Joseph S. Myers jos...@codesourcery.com

[RFC][PATCH] Remove SLOW_BYTE_ACCESS

2017-11-17 Thread Wilco Dijkstra
Remove SLOW_BYTE_ACCESS given it's confusing, badly named, badly documented and used incorrectly. Although most targets define it as 1, there are several targets which confuse it (based on comments next to it) and set it to 0 since the name obviously implies it should be 0 when byte accesses are