[PATCH] extracting swapStruct into include/llvm/Support/MachO.h (no functional change)
Duncan P. N. Exon Smith
dexonsmith at apple.com
Thu Jul 17 09:54:50 PDT 2014
> On 2014-Jul-09, at 03:19, Artyom Skrobov <Artyom.Skrobov at arm.com> wrote:
> Thank you Duncan
>>> While committing this, I realised that the instantiations of swapStruct()
>>> tools/lld/lib/ReaderWriter/MachO/MachONormalizedFileBinaryUtils.h are
>>> identical to the private implementations in
>>> Should perhaps these swapStruct()'s be extracted into
>>> include/llvm/Support/MachO.h ?
>>> Attaching the patch for review.
>> Sorry, I've lost context on this -- can you re-attach the patch?
> No problem -- attaching that patch again.
This LGTM, but I'm not sure I'm the right one to review it.
Nick, could you take a look at Artyom's patch (reattached here)? Is
this direction okay?
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 20361 bytes
Desc: not available
More information about the llvm-commits