joker.eph added inline comments.
================
Comment at: lib/CodeGen/CodeGenAction.cpp:660-663
@@ -660,1 +659,6 @@
+ OwnsVMContext(!_VMContext) {
+#ifdef NDEBUG
+ _VMContext.setDiscardValueNames(true);
+#endif
+}
----------------
chandlerc wrote:
> I'm surprised this isn't a driver-side NDEBUG default flip? Can this still be
> overridden with a flag?
It's gonna be next patch: plumbing a cc1 flag somehow. Right now this is
minimum patch to maintain existing behavior in clang while adapting to the
IRBuilder API in LLVM.
http://reviews.llvm.org/D18024
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
