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