<div dir="rtl"><div dir="ltr">e-mail and phab crossover.. what I meant was only moving the added code lines to CodeGenModule::Release(), not changing the module flag mechanism of passing the value, like this:</div><div dir="ltr"><br></div><div dir="ltr"><br></div><div dir="ltr"><div dir="ltr">  if (CodeGenOpts.Autolink &&</div><div dir="ltr">      (Context.getLangOpts().Modules || !LinkerOptionsMetadata.empty())) {</div><div dir="ltr">    EmitModuleLinkOptions();</div><div dir="ltr">  }</div><div dir="ltr"><br></div><div dir="ltr"><b>  // Record mregparm value now so it is visible through rest of codegen.</b></div><div dir="ltr"><b>  if (Context.getTargetInfo().getTriple().getArch() == llvm::Triple::x86)</b></div><div dir="ltr"><b>    getModule().addModuleFlag(llvm::Module::Error, "NumRegisterParameters",</b></div><div dir="ltr"><b>                              CodeGenOpts.NumRegisterParameters);</b></div><div dir="ltr"><br></div><div dir="ltr">  if (CodeGenOpts.DwarfVersion) {</div><div dir="ltr">    // We actually want the latest version when there are conflicts.</div><div dir="ltr">    // We can change from Warning to Latest if such mode is supported.</div><div dir="ltr">    getModule().addModuleFlag(llvm::Module::Warning, "Dwarf Version",</div><div dir="ltr">                              CodeGenOpts.DwarfVersion);</div><div dir="ltr">  }</div><div><br></div></div></div><br><div class="gmail_quote"><div dir="rtl">‫בתאריך יום ד׳, 29 במרץ 2017 ב-22:54 מאת ‪Yaron Keren via Phabricator‬‏ <‪<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>‬‏>:‬<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">yaron.keren added a comment.<br class="gmail_msg">
<br class="gmail_msg">
No intention of any major work intended... just moved the code (locally) to Release and regression tests were still passing.<br class="gmail_msg">
<br class="gmail_msg">
I must be missing something - isn't the module flag is used in X86ISelLowering which is called after CodeGenModule::Release() is done?<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
Repository:<br class="gmail_msg">
  rL LLVM<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D27051" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D27051</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>