rjmccall added a comment.

This looks fantastic, thanks for doing all that.  Approved with the one minor 
change.


================
Comment at: lib/Sema/SemaPseudoObject.cpp:175
@@ -145,1 +174,3 @@
+            PSE->getType(), PSE->getValueKind(), PSE->getObjectKind(),
+            PSE->getRBracketLoc());
       }
----------------
One extremely minor tweak: could you organize this like the rest of the 
pseudo-object expressions, checking for it above and calling a method to 
rebuild it?


http://reviews.llvm.org/D13336



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

Reply via email to