lebedev.ri added a comment. In D44823#1375654 <https://reviews.llvm.org/D44823#1375654>, @mclow.lists wrote:
> In D44823#1375590 <https://reviews.llvm.org/D44823#1375590>, @mclow.lists > wrote: > > > I just tried this (on Compiler Explorer) using LLVM 7, and the code for my > > original test in https://bugs.llvm.org/show_bug.cgi?id=35637 is now optimal. > > > Pilot error - it's still the same. Not too surprising, i don't recall any of the `memset` llvm differentials progressing. Though i think there was a consensus in some one of them that the transform itself is valid. Not sure which patch was the closest one to completion, or what it would take though. Repository: rCXX libc++ CHANGES SINCE LAST ACTION https://reviews.llvm.org/D44823/new/ https://reviews.llvm.org/D44823 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits