================
@@ -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

Reply via email to