[patch] Reject alias to declarations in ELF and COFF
Eric Christopher
echristo at gmail.com
Wed Nov 13 14:23:58 PST 2013
Looks like the majority of the change here is updating testcases
after? Can you elaborate a bit?
-eric
On Thu, Oct 31, 2013 at 9:58 AM, Rafael EspĂndola
<rafael.espindola at gmail.com> wrote:
> Since it looks like MachO will be getting support for aliases with
> N_INDR, this patch just rejects aliases to declarations on ELF and
> COFF.
>
> The use of report_fatal_error is probably OK, since it shows a problem
> in the FE producing the alias. Clang for example rejects this
> already.
>
> Another option would be to add target information to the verifier, but
> that looks odd.
>
> Cheers,
> Rafael
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
More information about the llvm-commits
mailing list