[fpc-devel] More peephole
lazarus at mfriebe.de
Wed Jan 22 00:27:16 CET 2014
On 21/01/2014 21:28, Florian Klämpfl wrote:
> Am 20.01.2014 01:18, schrieb Martin:
>> It used
>> (taicpu(p).oper^.reg<>taicpu(hp1).oper^^.ref^.base) and
>> (taicpu(p).oper^.reg<>taicpu(hp1).oper^^.ref^.index) then
>> but should only compare the supregister part
>> I replaced that
>> not(RegInOp(getsupreg(taicpu(p).oper^.reg),taicpu(hp1).oper^)) then
>> uncommented, and tested.
>> It does catch a big lot of occurrences.
> Can you post some example code? It might be worth to think about
> improving this already in at the node level.
I will try to find some. (I just enabled it, and put a writeln in there,
to see, if it was triggered. Then run the tests and buli Lazarus.
In the meantime, what about the other additions/changes?
I already wrote code for them, and mailed it.
So what I need to know: How to best go on to get them accepted and into
One problem is, if I create 3 or 4 patches, and report them, only one
will work. The others may then conflict. Or I need to create them, based
on each other, and only applicable in the right order.
I can do all in one (if that is preferred)...
More information about the fpc-devel