[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