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

Dávid Bolvanský via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 15 06:13:17 PST 2019


xbolva00 added a comment.

> 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.

Also, you will pessimize codegen for no real reason for AMD.


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

https://reviews.llvm.org/D70157





More information about the cfe-commits mailing list