[PATCH] D12916: [Static Analyzer] Use generics related information to infer dynamic types.

Anna Zaks via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 17 12:27:53 PDT 2015


zaks.anna added inline comments.

================
Comment at: lib/StaticAnalyzer/Checkers/DynamicTypePropagation.cpp:871
@@ +870,3 @@
+  if (RetRegion) {
+    State = setDynamicTypeInfo(State, RetRegion, ResultType,
+                               /*CanBeSubclass=*/true);
----------------
Would it be possible for this to reset the type to a less specific type? (What happens if the method was inlined and we already inferred the value of the returned symbolic region.)
Same question for where we update the type in  dynamicTypePropagationOnCasts.

This also introduces a situation where both tables track the type. Maybe we should add a TODO here?




http://reviews.llvm.org/D12916





More information about the cfe-commits mailing list