================ @@ -5153,6 +5156,7 @@ bool FunctionEffect::shouldDiagnoseFunctionCall( // ===== void FunctionEffectSet::Profile(llvm::FoldingSetNodeID &ID) const { + ID.AddInteger(size()); if (PImpl) for (const auto &Effect : *PImpl) ID.AddInteger(llvm::to_underlying(Effect.kind())); ---------------- Sirraide wrote:
Yeah, so long as this loop always adds exactly `size()` many effects, it should be fine. https://github.com/llvm/llvm-project/pull/84983 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits