<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Nov 3, 2021 at 2:37 AM Maksim Panchenko <<a href="mailto:maks@fb.com">maks@fb.com</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">





<div lang="EN-US" style="overflow-wrap: break-word;">
<div class="gmail-m_5000589816441763755WordSection1">
<p class="MsoNormal">> Making sure things are LLVM coding style, format, etc are pretty<u></u><u></u></p>
<p class="MsoNormal">> important to me. flang was a useful pilot here in a lot of ways and some<u></u><u></u></p>
<p class="MsoNormal">> of the requests for fixups still don't really seem to have happened so<u></u><u></u></p>
<p class="MsoNormal">> making sure that happens before is pretty important to me.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Following someone's footsteps has its drawbacks too :) Fair enough. We<u></u><u></u></p>
<p class="MsoNormal">continue to develop BOLT and have plans for more tight integration with<u></u><u></u></p>
<p class="MsoNormal">LLVM, such as sharing the common binary IR. Having a clean and<u></u><u></u></p>
<p class="MsoNormal">easy-to-read codebase will be highly beneficial to the community.<u></u><u></u></p>
<p class="MsoNormal"><u></u> </p></div></div></blockquote><div><br></div><div>Excellent. Looking forward to the code review!</div><div> </div><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_5000589816441763755WordSection1"><p class="MsoNormal"><u></u></p>
<p class="MsoNormal">>> We haven't made the final decision on including BOLT in<u></u><u></u></p>
<p class="MsoNormal">>> LLVM_ALL_PROJECTS, but we are ready to make BOLT the default project in<u></u><u></u></p>
<p class="MsoNormal">>> the initial merge commit. Currently, we do not support building on<u></u><u></u></p>
<p class="MsoNormal">>> Windows; thus, the project will be automatically disabled with the<u></u><u></u></p>
<p class="MsoNormal">>> warning when building on unsupported platforms. As we anticipate the<u></u><u></u></p>
<p class="MsoNormal">><u></u> <u></u></p>
<p class="MsoNormal">> Oh? Build or work? And why :)<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">The Windows build was disabled due to the usage of threads, IIRC.<u></u><u></u></p>
<p class="MsoNormal">Although, we don't use them directly, only via the C++11 standard<u></u><u></u></p>
<p class="MsoNormal">library. I don't have access to a windows box to test it myself. If the<u></u><u></u></p>
<p class="MsoNormal">build issues are resolved, I don't see why BOLT can't be used to<u></u><u></u></p>
<p class="MsoNormal">optimize ELF binaries on windows.<u></u><u></u></p>
<p class="MsoNormal"><u></u> </p></div></div></blockquote><div><br></div><div>Makes sense. Might be good to just disable threads on windows rather than disabling the build also.</div><div><br></div><div>Thanks for the response!</div><div><br></div><div>-eric</div><div> </div><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_5000589816441763755WordSection1"><p class="MsoNormal"><u></u></p>
<p class="MsoNormal">Thanks!<u></u><u></u></p>
<p class="MsoNormal">Maksim<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">On 11/2/21, 3:15 PM, "Eric Christopher" <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:<u></u><u></u></p>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">Hi Maksim,<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">I think this sounds great in general. <u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">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.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<p class="MsoNormal">One inline question:<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">We haven't made the final decision on including BOLT in<u></u><u></u></p>
<p class="MsoNormal">LLVM_ALL_PROJECTS, but we are ready to make BOLT the default project in<u></u><u></u></p>
<p class="MsoNormal">the initial merge commit. Currently, we do not support building on<u></u><u></u></p>
<p class="MsoNormal">Windows; thus, the project will be automatically disabled with the<u></u><u></u></p>
<p class="MsoNormal">warning when building on unsupported platforms. As we anticipate the<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Oh? Build or work? And why :)<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Thanks and good luck!<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">-eric <u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div>

</blockquote></div></div>