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

Reply via email to