LGTM

================
Comment at: lib/CodeGen/CodeGenModule.h:1198-1199
@@ -1197,4 +1197,4 @@
 
-  /// Determine if the given decl can be emitted lazily; this is only relevant
-  /// for definitions. The given decl must be either a function or var decl.
-  bool MayDeferGeneration(const ValueDecl *D);
+  /// Determine if the given definition must be emitted, or can otherwise be
+  /// emitted lazily.
+  bool MustBeEmitted(const ValueDecl *D);
----------------
This sounds like "determine whether either (a) the definition must be emitted 
or (b) the definition can be emitted lazily" whereas I think you mean 
"determine whether the definition must be emitted; if this returns \c false, 
the definition can be emitted lazily if it's used".

http://reviews.llvm.org/D6674

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/



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

Reply via email to