[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