[PATCH] D70157: Align branches within 32-Byte boundary

Fri Nov 15 06:40:17 PST 2019

davezarzycki added a comment.

In D70157#1747510 <https://reviews.llvm.org/D70157#1747510>, @xbolva00 wrote:

> > Even though core2 isn't affected by the erratum, core2 code can run on CPUs that do have the bug (and core2 is a popular target for code that needs to run "everywhere"), therefore all target CPUs that predate a hardware fix really
>
> So perf of all code/generic codegen will go down. This is not very acceptable. -1 for making this enabled by default for generic codegen.

I'm okay with this not being the default if we document somewhere that LLVM assumes that machines have up to date microcode installed.

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70157/new/

https://reviews.llvm.org/D70157