================
@@ -117,7 +117,7 @@ class CIRGenFunctionInfo final
                       RequiredArgs required, CanQualType resultType,
                       llvm::ArrayRef<CanQualType> argTypes) {
     id.AddBoolean(info.getNoReturn());
-    id.AddBoolean(required.getOpaqueData());
+    id.AddInteger(required.getOpaqueData());
----------------
andykaylor wrote:

This was the actual source of the bug. Adding this as a boolean rather than an 
integer was causing us to incorrectly match a cached `CIRGenFunctionInfo` in 
some cases. This was why one of the examples in the bug report failed while the 
other example worked. The failing case had a function info object that matched 
except for not being variadic.

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

Reply via email to