[PATCH] Delete AArch64II::MO_CONSTPOOL.

Tim Northover via llvm-commits llvm-commits at lists.llvm.org
Fri May 27 20:36:03 PDT 2016


On 27 May 2016 at 04:57, Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:
> I don't think so. It is a really odd restriction to not support got
> entries in some cases. I am actually tempted to ask for this to be a
> local change, after all, not having this makes for better results for
> other linkers and easier to maintain code for us.
>
> A rebased patch is attached. Tim, what do you think? If we have to
> keep it I will move it behind a flag.

I think I agree with removing it, though I could be persuaded towards
timed deprecation (via a flag) if anyone actually trying to use
armlink with OSS Clang pipes up.

I distinctly remember thinking the GOT solution is something that
should work in a sane world when I originally chose it. Let's move
towards that sane world!

Tim.


More information about the llvm-commits mailing list