[patch] Reject alias to declarations in ELF and COFF

Bob Wilson bob.wilson at apple.com
Wed Nov 13 15:50:28 PST 2013

On Nov 13, 2013, at 3:32 PM, Rafael Espíndola <rafael.espindola at gmail.com> wrote:

> On 13 November 2013 18:13, Bob Wilson <bob.wilson at apple.com> wrote:
>> I’m not familiar with N_INDR, but if it is not supported by the latest shipping version of OS X, then we shouldn’t have anything in LLVM that depends on it.  Even then, it would be best to have a configure check to enable it.
> It is not shipping. Currently llvm miscompiles an alias to a
> declaration. What this patch does is produce an error for ELF and COFF
> and leaves the status quo for MachO. I can change it to produce an
> error on MachO too for now if you want.

Yes, I think an error would be preferable to miscompiling.

More information about the llvm-commits mailing list