[PATCH] D96346: [NFC][PPC] Refactor TOC representation to allow several entries for the same symbol

Tue Feb 16 23:21:02 PST 2021

dyung added a comment.

In D96346#2567215 <https://reviews.llvm.org/D96346#2567215>, @jankratochvil wrote:

> Unfortunately it fails to build with GCC which LLVM still tries to stay compatible with:
> http://lab.llvm.org:8014/#/builders/16/builds/2211
> gcc-11.0.0-0.19.fc34.x86_64
> gcc-10.2.1-9.fc33.x86_64
> llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp:86:8: error: explicit specialization of 'template<class T> struct llvm::DenseMapInfo' outside its namespace must use a nested-name-specifier [-fpermissive]
>
>   86 | struct DenseMapInfo<std::pair<const MCSymbol *, MCSymbolRefExpr::VariantKind>> {
>      |        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> I have filed it to an existing GCC bugreport: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92598

I have committed 0e3d7e61867d69721b28e557272bdf4b66010327 <https://reviews.llvm.org/rG0e3d7e61867d69721b28e557272bdf4b66010327> in order to unblock the build on gcc builders.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96346/new/

https://reviews.llvm.org/D96346