
<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Hi Benjamin,<div><br></div><div>Attached is the rebased patch.</div><div><br></div><div>I’ve kept your if/else construction for the setting of the mask, but I can apply the array of array approach of my initial proposal if you prefer.</div><div><br></div><div>Note that like the original patch, this patch also corrects the low <-> high inversion.</div><div>This result in a diff on idiv test case for SSE variant that I believe is correct.</div><div>Indeed, there is no reason that we see the fix up of pmuludq in that test, since we are just reading the low bits!</div><div><br></div><div>Is it OK to commit?</div><div><br></div><div>Thanks,</div><div>-Quentin</div><div></div></body></html>