rsmith added a comment.

In D68896#1778193 <https://reviews.llvm.org/D68896#1778193>, @kianm wrote:

> Hi, I am still seeing problems with this assertion. Could we please get a 
> fix? I've posted the reduced test case and reproducible command on this 
> Phabricator patch.


Are you still seeing problems? If so, please can you file a bug report? I can't 
reproduce this failure with a recent Clang; I get

  <stdin>:14:3: error: ambiguous partial specializations of 'Base<A, '\x01'>'
    Base<A, C>::f();
    ^
  <stdin>:7:7: note: partial specialization matches [with T = A]
  class Base<T, 1> { };
        ^
  <stdin>:10:7: note: partial specialization matches [with T = A, I = 1]
  class Base<T, I> { };
        ^

but no assertion failure for the posted testcase.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68896



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D68896: P... Richard Smith - zygoloid via Phabricator via cfe-commits

Reply via email to