LGTM

================
Comment at: lib/Sema/SemaCast.cpp:1786
@@ +1785,3 @@
+      // haven't yet.
+      Self.RequireCompleteType(OpRange.getBegin(), SrcType, 0);
+      Self.RequireCompleteType(OpRange.getBegin(), DestType, 0);
----------------
OK, seems like Sema is supposed to RequireCompleteType before asking for the 
size.


http://llvm-reviews.chandlerc.com/D2548

BRANCH
  PR18479

ARCANIST PROJECT
  clang
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to