<div dir="ltr">It's not the clang side that's the problem. The llvm Module IR needs this info for libcall generation and the IR should probably not depend on CodeGenOpts. There may be another way to plumb it but it's not immediately obvious to me. <div><br></div><div>-Nirav</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 29, 2017 at 3:44 PM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Wed, Mar 29, 2017 at 12:30 PM, Nirav Davé <span dir="ltr"><<a href="mailto:niravd@google.com" target="_blank">niravd@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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></blockquote><div><br></div></span><div>Are you sure it requires any complex plumbing? CodeGenOpts is already available in CodeGenModule::Release. I think you can go ahead and move the code to set the module flag for consistency. <br></div></div></div></div>
</blockquote></div><br></div>