[PATCH] D64041: [analyzer] exploded-graph-rewriter: Add support for dynamic types.

Csaba Dabis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 1 15:24:42 PDT 2019


Charusso accepted this revision.
Charusso marked an inline comment as done.
Charusso added a comment.
This revision is now accepted and ready to land.

Nice, the end of that huge work is getting closer. Thanks!



================
Comment at: clang/utils/analyzer/exploded-graph-rewriter.py:229
+                for t in json_ps['dynamic_types']]) \
+            if json_ps['dynamic_types'] is not None else None
+
----------------
Hm, I would avoid that over-complicated code, but because most likely it is remain like that for the next decade, I see no problem here.


Repository:
  rC Clang

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

https://reviews.llvm.org/D64041





More information about the cfe-commits mailing list