<div dir="ltr"><div>Thanks for reviewing this. </div><div><br></div><div>The new constructor is to avoid initializing the "OS" field (which is not used by the empty BackendConsumer. Setting this field will affect the output). I will add a comment there.</div><div><br></div><div>-Rong</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jan 13, 2020 at 4:50 PM Teresa Johnson via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">tejohnson accepted this revision.<br>
tejohnson added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
LGTM. One more request for a comment below that I forgot to add earlier.<br>
<br>
<br>
<br>
================<br>
Comment at: clang/lib/CodeGen/CodeGenAction.cpp:154<br>
     }<br>
+    BackendConsumer(BackendAction Action, DiagnosticsEngine &Diags,<br>
+                    const HeaderSearchOptions &HeaderSearchOpts,<br>
----------------<br>
Add comment as to what specific case this new constructor is for.<br>
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D72523/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D72523/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D72523" rel="noreferrer" target="_blank">https://reviews.llvm.org/D72523</a><br>
<br>
<br>
<br>
</blockquote></div></div>