Page MenuHomePhabricator

[Codegen] Updated test for D66158
ClosedPublic

Authored by xbolva00 on Tue, Aug 13, 2:41 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

xbolva00 created this revision.Tue, Aug 13, 2:41 PM
Herald added a project: Restricted Project. · View Herald TranscriptTue, Aug 13, 2:41 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript

Might not be the best test given that this should arguably be a memset not a memcpy.

hm.. memset?

xbolva00 updated this revision to Diff 214940.Tue, Aug 13, 3:12 PM

Explicit memcpy

jdoerfert accepted this revision.Tue, Aug 13, 3:23 PM

hm.. memset?

When I say memset, here and in the other review, I mean memmove.

Explicit memcpy

thx. LGTM.

This revision is now accepted and ready to land.Tue, Aug 13, 3:23 PM
xbolva00 updated this revision to Diff 215039.Wed, Aug 14, 1:18 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptWed, Aug 14, 1:33 AM