<div dir="ltr">Doing so would require non-trivial plumbing of NumRegParameters through the llvm IR. <div><br></div><div>That said, I don't believe there's any reason that we cannot move all the flag declarations up to the end of CodeGenModule::CodeGenModule so they're all in one place again. <div><div><br></div><div>-Nirav</div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 29, 2017 at 3:13 PM, Yaron Keren via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">yaron.keren added a comment.<br>
<br>
Most other module flags are added at CodeGenModule::Release().  For consistency, could this code be in CodeGenModule::Release() as well?<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D27051" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D27051</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div>