ABataev added inline comments.

================
Comment at: include/clang/Driver/Action.h:493
@@ -491,1 +492,3 @@
 
+class OffloadUnbundlingJobAction : public JobAction {
+  void anchor() override;
----------------
'final'?

================
Comment at: include/clang/Driver/Action.h:499
@@ +498,3 @@
+  /// this unbundling action.
+  struct DependingActionInfoTy {
+    /// \brief The tool chain of the depending action.
----------------
1. 'final'?
2. Default initializers for fields

================
Comment at: include/clang/Driver/Types.h:78
@@ +77,3 @@
+  /// compilation phase is a preprocessor one.
+  bool isSrcFile(ID Id);
+
----------------
'const'?


http://reviews.llvm.org/D21853



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

Reply via email to