[PATCH] D83283: [Attributor] AAPotentialValues Interface

[Shinji Okumura via Phabricator via llvm-commits]

okura added a comment.

In D83283#2189444 <https://reviews.llvm.org/D83283#2189444>, @thakis wrote:

> This causes:
>
> llvm/lib/Transforms/IPO/AttributorAttributes.cpp:56:49: error: explicit specialization of 'MaxPotentialValues' after instantiation
> unsigned llvm::PotentialConstantIntValuesState::MaxPotentialValues = 0;
>
>   ^
>
> ../../llvm/lib/Transforms/IPO/AttributorAttributes.cpp:52:57: note: implicit instantiation first required here
>
>   cl::location(llvm::PotentialConstantIntValuesState::MaxPotentialValues),
>                                                       ^

I confirmed that I can build successfully with g++ but fail with clang.
Should I revert the commit right away?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83283