search for: d35140

David/Johan, I would love to claim victory, but I don't think that D34901 catches this case. However, I got interested and threw this together quickly: https://reviews.llvm.org/D35140. This does catch the below case. If people are interested I can add test cases and submit for formal review. FWIW, it does hit about 1/3 of all of the SPEC benchmarks. I haven't done any performance analysis to see if it matters, however. Chad On 7/7/2017 11:52 AM, Davide Italiano v...

...4901 catches this >> case. >> > Hi Chad, thanks for taking another look at this. > Maybe I didn't bisect correctly. Apologies. Anyway, more fun for us. Yes, indeed. More fun! >> However, I got interested and threw this together quickly: >> https://reviews.llvm.org/D35140. >> >> This does catch the below case. If people are interested I can add test >> cases and submit for formal review. FWIW, it does hit about 1/3 of all of >> the SPEC benchmarks. I haven't done any performance analysis to see if it >> matters, however. >>...

On Thu, Jul 6, 2017 at 7:00 AM, Davide Italiano <davide at freebsd.org> wrote: > On Wed, Jul 5, 2017 at 3:59 PM, Johan Engelen via llvm-dev > <llvm-dev at lists.llvm.org> wrote: > > Hi all, > > I just found an optimization regression regarding simple > > dataflow/constprop analysis: > > https://godbolt.org/g/Uz8P7t > > > > This code >