No functional changes. Just created wrapper classes around the 3 and 4 reg mult and mac instruction classes.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
Hi Sam,
Sorry it took so long, this is a complicated patch. :)
All in all, this is a great patch. Removes a lot of duplication and makes it less error prone. Thanks for doing this work.
I couldn't see anything wrong with it, though I may have missed in the huge list of changes. I'm assuming this passes all tests (internal and external), so, LGTM.
Thanks!
--renato
Comment Actions
Original commit was reverted due to the thumb umaal description being incorrect and there was no test case to catch it. Updated patch corrects the bug and adds testing for t2UMAAL
Comment Actions
Hi Renato,
Are you able to see the change clearly now? I'm not really sure how to use this interface so can't tell if you can see the incremental change.
cheers,
sam