On 6/15/23 09:30, Manolis Tsamis wrote:
Thanks for reporting. I also noticed this while reworking the
implementation for v2 and I have fixed it among other things.
Sounds good. I stumbled across another problem while testing V2.
GEN_INT can create a non-canonical integer constant (and one might
legitimately wonder if we should eliminate GEN_INT). The specific case
I ran into was something like 0xfffffff0 for an SImode value on a 64bit
host. That should have been 0xfffffffffffffff0 to be canonical.
The right way to handle this these days is with gen_int_mode. You
should replace the two calls to GEN_INT with gen_int_mode (new_offset, mode)
Still testing the new variant...
jeff
