This is an archive of the discontinued LLVM Phabricator instance.

Fix big-endian byte swap for WindowsResourceDumper.
AcceptedPublic

Authored by mnbvmar on Sep 21 2017, 1:05 PM.

Details

Reviewers
zturner
Summary

Previous patch incorrectly implemented swapping UTF16 byte order. Hope this should work alright, but I want to make sure beforehand.

Diff Detail

Event Timeline

mnbvmar created this revision.Sep 21 2017, 1:05 PM
zturner accepted this revision.Sep 21 2017, 1:32 PM
This revision is now accepted and ready to land.Sep 21 2017, 1:32 PM
seurer added a subscriber: seurer.Sep 21 2017, 1:48 PM

I tried this patch on powerpc64 (BE) and it fixed the problem.