Re: [PATCH v1 00/18] SSA conversion, the classical way

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 25 March 2018 at 23:03, Dibyendu Majumdar <mobile@xxxxxxxxxxxxxxx> wrote:
> On 25 March 2018 at 22:43, Dibyendu Majumdar <mobile@xxxxxxxxxxxxxxx> wrote:
>> On 20 March 2018 at 21:52, Dibyendu Majumdar <mobile@xxxxxxxxxxxxxxx> wrote:
>>> On 20 March 2018 at 21:49, Luc Van Oostenryck
>>> <luc.vanoostenryck@xxxxxxxxx> wrote:
>>>> On Tue, Mar 20, 2018 at 09:07:47PM +0000, Dibyendu Majumdar wrote:
>>>>> On 20 March 2018 at 00:52, Luc Van Oostenryck
>>>>> <luc.vanoostenryck@xxxxxxxxx> wrote:
>>>>> > This series replace the current SSA conversion by one using
>>>>> > the 'classical' way, via the iterated dominance frontier.
>>>>> >
>>>>>
>>>>> Thank you for posting this patch series. I would like to test it. I
>>>>> will apply my LLVM fixes against your tree so that I can run the set
>>>>> of tests I have prepared. This will take a bit of time but I will
>>>>> report back here on results.
>>>>
>>
>> I have done some initial work on this. My changes are in:
>> https://github.com/dibyendumajumdar/sparse-dev/commits/ssa-v1. Some
>> more work is needed as this is still preliminary.
>>
>> So far 65 out of 85 tests in
>> (https://github.com/dibyendumajumdar/sparse-testing) pass. I haven't
>> yet investigated the failures - they could be due to issues in my
>> merge.
>>
>
> Improved to 70 out of 85 after removing the store instruction for
> initializing aggregates.
>


Luc,

BTW you have said several times that you don't care about the LLVM
backend .... I do, and if you merge my version into your tree, I will
be happy to maintain it.

Thanks and Regards
Dibyendu
--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Newbies FAQ]     [LKML]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Trinity Fuzzer Tool]

  Powered by Linux