[PATCH] Some code improvements (no functional change)

Chandler Carruth chandlerc at google.com
Thu Jun 12 08:52:01 PDT 2014


And now I've found it. Ignore me... sorry, I don't know why my email client
decided to fork the thread at Duncan's email adding others to the CC...


On Thu, Jun 12, 2014 at 4:50 PM, Chandler Carruth <chandlerc at google.com>
wrote:

> No, i mean, I don't even see the original patch or patch email at all.
> This seems to be an email client problem on my end, but it would help to
> re-post the patch at this point I suspect.
>
>
> On Thu, Jun 12, 2014 at 4:46 PM, Duncan P. N. Exon Smith <
> dexonsmith at apple.com> wrote:
>
>>
>> > On 2014 Jun 12, at 08:42, Chandler Carruth <chandlerc at google.com>
>> wrote:
>> >
>> > I'm lacking context sadly. =/
>>
>> Idea is:
>>
>>   - SwapByteOrder takes args by value and returns swapped bytes.
>>   - Some users want to swap in place.
>>   - So, rename SwapByteOrder => getSwappedBytes.
>>   - Then, add swapByteOrder (which takes and modifies a reference).
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140612/c4e97888/attachment.html>


More information about the llvm-commits mailing list