
<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div>Okay, here is a newer revision of the patch. Changes:</div><div><br></div><div>1) I only registered for i32s. I think there were probably some issues with i64s and i32 is going to be the more common case on ARM.</div><div>2) I updated all the tests to use regexps and track variables.</div><div>3) I spent some time trying to track through and get it to go to ADDS/SUBS, but in the end that proved to be more work than I thought it would be due to the differences between ARM and ARM64 in how the CPSR is handled, and the ARM models CMPs directly (where as ARM64 converts them to SUBS), and the fact that ARM backend only has working plumbing for CMP, not CMN. This patch is substantially better behavior than we currently have, and I would rather get it in now than wait trying to eek out that last bit of optimization.</div><div><br></div><div>Louis</div><div><br></div></body></html>