[PATCH] D72225: Align branches within 32-Byte boundary(Prefix padding)

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 13 14:48:53 PST 2020


MaskRay added a comment.

@skan @craig.topper This patch introduces an unknown bug. I applied Diff 15 (https://reviews.llvm.org/D72225?id=237569) + D72463 <https://reviews.llvm.org/D72463> (clangDriver) + a local patch which enables `-mbranches-within-32B-boundaries` by default, then randonly picked 1000 tests and 40 failed. I will try providing a reproduce.

NOP padding alone seems good.


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

https://reviews.llvm.org/D72225





More information about the llvm-commits mailing list