<div dir="ltr"><div>Hi Maksim,</div><div><br></div><div>I think this sounds great in general. </div><div><br></div><div>Making sure things are LLVM coding style, format, etc are pretty important to me. flang was a useful pilot here in a lot of ways and some of the requests for fixups still don't really seem to have happened so making sure that happens before is pretty important to me.</div><div><br></div>One inline question:<div><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div lang="EN-US" style="overflow-wrap: break-word;"><div class="gmail-m_-3415971134563114844WordSection1">
<p class="MsoNormal"><span style="font-size:11pt">We haven't made the final decision on including BOLT in<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt">LLVM_ALL_PROJECTS, but we are ready to make BOLT the default project in<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt">the initial merge commit. Currently, we do not support building on<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt">Windows; thus, the project will be automatically disabled with the<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt">warning when building on unsupported platforms. As we anticipate the<u></u><u></u></span></p>
<p class="MsoNormal"><br></p></div></div></blockquote><div><br></div><div>Oh? Build or work? And why :)</div><div><br></div><div>Thanks and good luck!</div><div><br></div><div>-eric </div></div></div></div>