================ @@ -87,6 +87,9 @@ class alignas(const Formula *) Formula { ArrayRef<const Formula *> Operands, unsigned Value = 0); + // Parse Formulas using Arena rather than caling this function directly. ---------------- martinboehme wrote:
```suggestion // Don't call this function directly. Use `Arena::parseFormula()` instead. ``` Nit: When I began reading the comment, "Parse Formulas using Arena" initially sounded like a description of what the function is doing. "Don't call directly" seems like the most helpful thing to put at the beginning. https://github.com/llvm/llvm-project/pull/66424 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits