<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=gb2312">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
span.EmailStyle18
{mso-style-type:personal-reply;
font-family:"Calibri",sans-serif;
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style>
</head>
<body lang="EN-US" link="#0563C1" vlink="#954F72" style="word-wrap:break-word">
<div class="WordSection1">
<p class="MsoNormal">> Making sure things are LLVM coding style, format, etc are pretty<o:p></o:p></p>
<p class="MsoNormal">> important to me. flang was a useful pilot here in a lot of ways and some<o:p></o:p></p>
<p class="MsoNormal">> of the requests for fixups still don't really seem to have happened so<o:p></o:p></p>
<p class="MsoNormal">> making sure that happens before is pretty important to me.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Following someone's footsteps has its drawbacks too :) Fair enough. We<o:p></o:p></p>
<p class="MsoNormal">continue to develop BOLT and have plans for more tight integration with<o:p></o:p></p>
<p class="MsoNormal">LLVM, such as sharing the common binary IR. Having a clean and<o:p></o:p></p>
<p class="MsoNormal">easy-to-read codebase will be highly beneficial to the community.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">>> We haven't made the final decision on including BOLT in<o:p></o:p></p>
<p class="MsoNormal">>> LLVM_ALL_PROJECTS, but we are ready to make BOLT the default project in<o:p></o:p></p>
<p class="MsoNormal">>> the initial merge commit. Currently, we do not support building on<o:p></o:p></p>
<p class="MsoNormal">>> Windows; thus, the project will be automatically disabled with the<o:p></o:p></p>
<p class="MsoNormal">>> warning when building on unsupported platforms. As we anticipate the<o:p></o:p></p>
<p class="MsoNormal">><o:p> </o:p></p>
<p class="MsoNormal">> Oh? Build or work? And why :)<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">The Windows build was disabled due to the usage of threads, IIRC.<o:p></o:p></p>
<p class="MsoNormal">Although, we don't use them directly, only via the C++11 standard<o:p></o:p></p>
<p class="MsoNormal">library. I don't have access to a windows box to test it myself. If the<o:p></o:p></p>
<p class="MsoNormal">build issues are resolved, I don't see why BOLT can't be used to<o:p></o:p></p>
<p class="MsoNormal">optimize ELF binaries on windows.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Thanks!<o:p></o:p></p>
<p class="MsoNormal">Maksim<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal">On 11/2/21, 3:15 PM, "Eric Christopher" <echristo@gmail.com> wrote:<o:p></o:p></p>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal">Hi Maksim,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">I think this sounds great in general. <o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></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.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<p class="MsoNormal">One inline question:<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">We haven't made the final decision on including BOLT in<o:p></o:p></p>
<p class="MsoNormal">LLVM_ALL_PROJECTS, but we are ready to make BOLT the default project in<o:p></o:p></p>
<p class="MsoNormal">the initial merge commit. Currently, we do not support building on<o:p></o:p></p>
<p class="MsoNormal">Windows; thus, the project will be automatically disabled with the<o:p></o:p></p>
<p class="MsoNormal">warning when building on unsupported platforms. As we anticipate the<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Oh? Build or work? And why :)<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Thanks and good luck!<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">-eric <o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</body>
</html>