================
@@ -306,6 +306,19 @@ equalAttrArgs(T A1, T A2, StructuralEquivalenceContext
&Context) {
return A1 == A2;
}
+// ParamIdx has an explicit specialization because it can be invalid (when
+// representing an optional parameter that was not specified). Two invalid
+// ParamIdx values compare equal; an invalid value is not equal to any valid
+// one. ParamIdx::operator== asserts both sides are valid, so we must guard
+// against the invalid case here before delegating to operator==.
+template <>
+bool equalAttrArgs<ParamIdx>(ParamIdx P1, ParamIdx P2,
+ StructuralEquivalenceContext &) {
+ if (!P1.isValid() || !P2.isValid())
----------------
shafik wrote:
The old maxim, if you write code as clever as you can you are by definition not
clever enough to debug it applies well all the time.
https://github.com/llvm/llvm-project/pull/199980
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
