llvm dev - Nov 2021 - Preparing BOLT for LLVM monorepo

Hi Maksim,

I think this sounds great in general.

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.

One inline question:

We haven't made the final decision on including BOLT
in
>
> LLVM_ALL_PROJECTS, but we are ready to make BOLT the default project in
>
> the initial merge commit. Currently, we do not support building on
>
> Windows; thus, the project will be automatically disabled with the
>
> warning when building on unsupported platforms. As we anticipate the
>
>
>
Oh? Build or work? And why :)

Thanks and good luck!

-eric
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20211102/2fed430a/attachment-0001.html>

> 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.
Following someone's footsteps has its drawbacks too :) Fair enough. We
continue to develop BOLT and have plans for more tight integration with
LLVM, such as sharing the common binary IR. Having a clean and
easy-to-read codebase will be highly beneficial to the community.
>> We haven't made the final decision on including BOLT in
>> LLVM_ALL_PROJECTS, but we are ready to make BOLT the default project in
>> the initial merge commit. Currently, we do not support building on
>> Windows; thus, the project will be automatically disabled with the
>> warning when building on unsupported platforms. As we anticipate the
>
> Oh? Build or work? And why :)
The Windows build was disabled due to the usage of threads, IIRC.
Although, we don't use them directly, only via the C++11 standard
library. I don't have access to a windows box to test it myself. If the
build issues are resolved, I don't see why BOLT can't be used to
optimize ELF binaries on windows.

Thanks!
Maksim

On 11/2/21, 3:15 PM, "Eric Christopher" <echristo at gmail.com>
wrote:

Hi Maksim,

I think this sounds great in general.

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.

One inline question:

We haven't made the final decision on including BOLT in
LLVM_ALL_PROJECTS, but we are ready to make BOLT the default project in
the initial merge commit. Currently, we do not support building on
Windows; thus, the project will be automatically disabled with the
warning when building on unsupported platforms. As we anticipate the


Oh? Build or work? And why :)

Thanks and good luck!

-eric

-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20211103/085f4ba2/attachment.html>