[PATCH] Rename getNoopForMachoTarget to getNoop

Brad Smith brad at comstyle.com
Mon Sep 15 20:36:32 PDT 2014


On 15/09/14 5:39 PM, Jim Grosbach wrote:
>
>> On Sep 15, 2014, at 12:50 PM, Brad Smith <brad at comstyle.com> wrote:
>>
>> On 15/09/14 2:45 PM, Rafael Espíndola wrote:
>>> On 15 September 2014 14:39, Brad Smith <brad at comstyle.com> wrote:
>>>> On 15/09/14 1:19 PM, Rafael Espíndola wrote:
>>>>>
>>>>> I think this is wrong. We should just produce empty functions for
>>>>> non-MachO.
>>>>
>>>>
>>>> Then send a diff to do it.
>>>
>>> Fixed. In the future, do *not* commit patches without approval.
>>
>> It was approved.
>
> Where? I just re-read the thread and don’t see it. Is there somewhere else I should be looking?

Since this diff wasn't even commited I'm assuming he wasn't talking 
about this diff thus looking through this thread won't be of any use.


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.





More information about the llvm-commits mailing list