This revision was automatically updated to reflect the committed changes.
Closed by commit rL323617: Change memcpy/memove/memset to have dest and source
alignment attributes. (authored by dneilson, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
rjmccall accepted this revision.
rjmccall added a comment.
Still LGTM.
Repository:
rC Clang
https://reviews.llvm.org/D41677
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
dneilson updated this revision to Diff 131685.
dneilson added a comment.
Herald added subscribers: niosHD, sabuasal, apazos, jordy.potman.lists,
simoncook, johnrusso, rbar, asb.
Rebaseline
Repository:
rC Clang
https://reviews.llvm.org/D41677
Files:
lib/CodeGen/CGBuilder.h
dneilson added a comment.
In https://reviews.llvm.org/D41677#966094, @rjmccall wrote:
> I'm glad to hear that progress is finally happening on this.
>
> The change to CGBuilder looks good to me. I'm going to take your word for it
> that the test changes are all just obvious updates; if there's
rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
I'm glad to hear that progress is finally happening on this.
The change to CGBuilder looks good to me. I'm going to take your word for it
that the test changes are all just obvious
dneilson created this revision.
dneilson added a reviewer: rjmccall.
Herald added subscribers: fedor.sergeev, kbarton, aheejin, sbc100, javed.absar,
nhaehnle, nemanjai, jyknight.
Upstream LLVM is changing the the prototypes of the @llvm.memcpy/memmove/memset
intrinsics. This change updates the