On 05/14/2018 10:55 PM, Jeff Law wrote:
> That does sound vaguely familiar.  Did we put autoinc notes on the stack
> pushes?

Not as far as I recall. I only see REG_ARGS_SIZE notes in the dumps.

> That makes me wonder if there is a latent bug though.  Consider pushing
> args to a pure function.  Could we then try to CSE the memory reference
> and get it wrong because we haven't accounted for the autoinc?

Can't know for sure but one would hope something would test for
side_effects_p.


Bernd

Reply via email to