================
@@ -3416,15 +3420,13 @@ static QualType checkBuiltinTemplateIdType(
     QualType OrigType = Converted[1].getAsType();
     // C++14 [inteseq.intseq]p1:
     //   T shall be an integer type.
-    if (!OrigType->isDependentType() && !OrigType->isIntegralType(Context)) {
+    if (!OrigType->isIntegralType(Context)) {
----------------
Serosh-commits wrote:

> I don't see a point in adding an assert for a condition you just checked and 
> bailed on just four lines above.

I added it because @ojhunt specifically requested for it but if u will see yeah 
it is not needed let me know we should add it or not 


https://github.com/llvm/llvm-project/pull/180407
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to