On 3/14/22 10:06 PM, Peter Bergner wrote: > On 3/14/22 8:24 PM, Segher Boessenkool wrote: >> You might want to name that common expression, "rtx addr = XEXP (op, 0);" >> or something. Dunno what is best > > Will do. > > >> Please put that new MEM_P code first, followed by a blank line, and only >> then do the SUBREG thing. As written it will allow subregs of mem. And >> the blank line is important of course ;-) > > Will do. > > >> Okay for trunk with those changes. Also okay for 10 and 11 after an >> appropriate soak period. Thanks!
Testing of the updates came back clean so I pushed the fix. I'll wait a few days before pushing the backports. Thanks! Peter
