[PATCH] D86223: [analyzer][z3] Use more elaborate z3 variable names in debug build

Gábor Horváth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Aug 22 08:43:40 PDT 2020


xazax.hun accepted this revision.
xazax.hun added a comment.

This is what I had in mind, thanks!



================
Comment at: clang/include/clang/StaticAnalyzer/Core/PathSensitive/SMTConv.h:335
+    llvm::raw_svector_ostream OS(Str);
+    OS << PRETTY_SYMBOL_KIND << ID;
+#undef PRETTY_SYMBOL_KIND
----------------
Maybe, in this case, it is cleaner to duplicate the line rather than introduce `PRETTY_SYMBOL_KIND`?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86223/new/

https://reviews.llvm.org/D86223



More information about the cfe-commits mailing list