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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to