On Wed, 12 Feb 2020 12:47:47 +0200, Mika Westerberg wrote: > On Wed, Feb 12, 2020 at 11:30:30AM +0100, Jean Delvare wrote: > > (The underlying question being: can I get rid of that ALIGN() > > altogether while fixing the gas->access_width misuse bug?) > > I think the ALIGN() was there just because I did not realize that > access_width is 3 and not 4 for 32-bit memory. So it is not needed. > > I actually have a patch series that should fix this and the other issue > you found (I found a couple of spare cycles in the morning) so if you > don't mind I'll submit them soon. Sure please do, I don't care about ownership, I only want these bugs to be fixed. I got myself side-tracked by another issue at work this morning. I'll be happy to review your changes. -- Jean Delvare SUSE L3 Support