[PATCH] D70157: Align branches within 32-Byte boundary
annita.zhang via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Nov 14 21:49:43 PST 2019
annita.zhang added a comment.
In D70157#1744243 <https://reviews.llvm.org/D70157#1744243>, @chandlerc wrote:
> Thanks for sending this out!
>
> I've made a minor comment on the flag structure, but my bigger question would be: can you summarize the performance hit and code size hit you've seen across some benchmarks? SPEC and the LLVM test suite would be nice, and maybe code size alone for some large binary like Chrome, Firefox, or Safari?
>
> I'd be particularly interested in comparing the performance & code size hit incurred by the suggested "fused,jcc,jmp" set compared to all (adding call, ret, and indirect). If there is a big drop in either, I'd love to know which of these incurs the large drop.
>
> Thanks,
> -Chandler
We are working on the performance and code size evaluation. Will update once it's available.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70157/new/
https://reviews.llvm.org/D70157
More information about the cfe-commits
mailing list