[clang] [clang][ASTVisitor] Visit `HoldingVar` from `BindingDecl`. (PR #117858)

[Yitzhak Mandelbaum via cfe-commits]

================
@@ -143,6 +143,15 @@ const Formula &getFormula(const ValueDecl &D, const Environment &Env) {
   return cast<BoolValue>(Env.getValue(D))->formula();
 }
 
+const BindingDecl *findBindingDecl(const char *Name, ASTContext &ASTCtx) {
----------------
ymand wrote:

why place this parameter second (vs first, like findValueDecl)? I don't have a preference, just curious about the difference.

https://github.com/llvm/llvm-project/pull/117858