On Tue, Mar 20, 2018 at 09:52:57PM +0000, Dibyendu Majumdar 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: > >> 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? Perfect then. Yes, my 'ssa' tree contains everything I used myself. > > - 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. OK. Thanks. -- Luc -- 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
