[PATCH] Rename getNoopForMachoTarget to getNoop

Rafael EspĂ­ndola rafael.espindola at gmail.com
Mon Sep 15 11:45:19 PDT 2014


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.

"This is not the correct way to fix it" is a pretty standard code
review. Please get used to it.

Cheers,
Rafael




More information about the llvm-commits mailing list