tejohnson accepted this revision.
tejohnson added a comment.
This revision is now accepted and ready to land.

LGTM. One more request for a comment below that I forgot to add earlier.



================
Comment at: clang/lib/CodeGen/CodeGenAction.cpp:154
     }
+    BackendConsumer(BackendAction Action, DiagnosticsEngine &Diags,
+                    const HeaderSearchOptions &HeaderSearchOpts,
----------------
Add comment as to what specific case this new constructor is for.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72523/new/

https://reviews.llvm.org/D72523



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

Reply via email to