On 07/13/11 16:01, Richard Earnshaw wrote: > On 07/07/11 21:02, Bernd Schmidt wrote: >> This corrects an error in store_multiple_operation. We're only >> generating the writeback version of the instruction on Thumb-1, so >> that's where we must make sure the base register isn't also stored. >> >> The ARMv7 manual is unfortunately not totally clear that this does in >> fact produce unpredictable results; it seems to suggest that this is the >> case only for the T2 encoding. Older documentation makes it clear. >> >> Tested on arm-eabi{,mthumb}. >> > > I agree that the wording here is unclear, but the pseudo code for the > decode makes the situation clearer, and does reflect what I really > believe to be the case. Put explicitly:
[...] I just remembered this patch. Your reply didn't actually comment on it, so - ok to install? bernd