[clang] [clang] disallow narrowing when matching template template parameters (PR #124313)

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 24 12:41:54 PST 2025


================
@@ -11682,6 +11683,7 @@ class Sema final : public SemaBase {
                         SmallVectorImpl<TemplateArgument> &SugaredConverted,
                         SmallVectorImpl<TemplateArgument> &CanonicalConverted,
                         CheckTemplateArgumentKind CTAK, bool PartialOrdering,
+                        bool PartialOrderingTTP,
----------------
erichkeane wrote:

Thats always the balance unfortunately.  I hate having 2 bools in a row like this, particularly when they are both SO similar in name/meaning.  I would still love an enum with 4 values, but I can wait to see if others are as concerned as I am, and disagree and commit if necessary.

https://github.com/llvm/llvm-project/pull/124313


More information about the cfe-commits mailing list