> This seems fine, your description "....does not always work." is a bit
> of a tease :) it would be nice to know _why_ it doesn't work, or at
> least a description of what problem you're seeing.
> 
As far as I can see, LRA doesn't handle very well the conditional execution 
patterns, as it expects conditional execution to happen after this step. Thus, 
some of those instructions are marked dead and removed later on.

> Also we seem to be missing a test, would it be possible to find one?
> If not then I guess we live without, but we should note that in the
> commit message.

This error is found by executing dg.exp testsuite with our port and -mlra 
option on.  As we speak, I am on the last 100m of testing our port having the 
LRA on. This bug being found like that.

I'll add this discussion to the commit message body.

Thank you,
Claudiu

Reply via email to