BOLT/LLVM? does not preserve prefixes on conditional branches about bolt HOT 2 OPEN

Yes, it looks similar to losing addr32 prefix: LLVM MC might be losing the prefix during disassembly, not setting it on MCInst.

On the other hand, these prediction prefixes are optional so we may want to strip them by default. What's the use case or perf effect here?

from bolt.

Thanks @aaupov

This is ignored by the current processors. The static prediction is NT (Not Taken) and the prefix is ignored. They are also mostly not generated (except when using special compiler flags and likely/unlikely macros). So striping them by default would be right.

With or without the hint, the BPU is updated when the branch is taken.

We are doing some early research work to mark common conditional branches with 3E (so the branch instruction is predicted Taken by the CPU). We wanted to use LBR or conditional branch taken to collect profile and then use BOLT to apply the hint.

For this the following we will need the following

1. The branch probability of the conditional branch (is there an easy way in BOLT to get to this or do we need to process the profile data to create this?)
2. To add the 3E prefix for conditional branches that are mostly taken ( I presume that adding the prefix will require some of the LLVM support?)
3. Write out the binary and preserve the 3E (looks like this is missing some LLVM support)

Appreciate any directions on this.

Thanks

--Suresh

from bolt.