On Mon, May 1, 2017 at 12:09 PM, Samuel Pitoiset
wrote:
> On 05/01/2017 05:59 PM, Ilia Mirkin wrote:
>>
>> I think this is off. It shouldn't matter what the code sequence is,
>> it's all representable. You need to teach replaceZero to not mess
>> things up for SHLADD's src(1).
>
>
> It's represent
On 05/01/2017 05:59 PM, Ilia Mirkin wrote:
I think this is off. It shouldn't matter what the code sequence is,
it's all representable. You need to teach replaceZero to not mess
things up for SHLADD's src(1).
It's representable but stupid to do it. We should keep the ADD there and
this also a
I think this is off. It shouldn't matter what the code sequence is,
it's all representable. You need to teach replaceZero to not mess
things up for SHLADD's src(1).
On Sat, Apr 29, 2017 at 12:01 PM, Samuel Pitoiset
wrote:
> Doing '(a << b) + c' when b is 0 is dumb, ADD should be used
> instead.
>
Doing '(a << b) + c' when b is 0 is dumb, ADD should be used
instead.
This fixes a compilation error with Alien Isolation because
src(1) is expected to be an immediate value, and the
replaceZero logic will transform 0 to $r63 (or $r255).
Signed-off-by: Samuel Pitoiset
Cc: "13.0 17.0 17.1"
---
