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

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

 



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:
>> Hi Luc,
>>
>> 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.
>
> Sure. Thank you.
> I must warn you of two things though:
> - most series I posted the last months and especially the last
>   two weeks are somehow needed to use and test this one.

I will apply my patches LLVM against your tree (not dmr_C) so all your
changes are present already in the tree I presume?

> - don't forget that for the reasons explained in the cover letter
>   (and a few others, less important), the SSA is still not correct
>   or at least is broken during simplify_loads() and branch rewritting.
>
> So, to stay 100% meaningful, you should drop all optimizations after
> ssa_convert() and insure that you don't have unreachable code in your
> input files.
>

Okay noted; I will report back anyway what the results are.

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