On Tue, Mar 17, 2015 at 10:26 AM, Peter Maydell <peter.mayd...@linaro.org> wrote: > On 17 March 2015 at 17:24, Greg Bellows <greg.bell...@linaro.org> wrote: >> On Tue, Mar 10, 2015 at 12:18 PM, Peter Maydell >> <peter.mayd...@linaro.org> wrote: >>> The A32 encoding of LDM distinguishes LDM (user) from LDM (exception >>> return) based on whether r15 is in the register list. However for >>> STM (user) there is no equivalent distinction. We were incorrectly >>> treating "r15 in list" as indicating exception return for both LDM >>> and STM, with the result that an STM (user) involving r15 went into >>> an infinite loop. Fix this; note that the value stored for r15 >>> in this case is the current PC regardless of our current mode. >>> >>> Signed-off-by: Peter Maydell <peter.mayd...@linaro.org> > >> Reviewed-by: Greg Bellows <greg.bell...@linaro.org> > > Thanks for the review, though this got committed to master yesterday :-) > > -- PMM
Oh missed that and I checked the archive and did not see any review-by and figured you wanted to get it in...
