[PATCH] D109817: [AggressiveInstCombine] Add `phi` nodes support to `TruncInstCombine`
Anton Afanasyev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 8 04:39:22 PST 2022
anton-afanasyev added a comment.
I've investigated small regressions, caused by this patch. The sizes are actually increased at the backend level, while translating llvm to machine code, for instanse:
> ll symtab.c.*
13769 symtab.c.base.ll
13681 symtab.c.exp.ll
8280 symtab.c.base.s
8394 symtab.c.exp.s
For the tests from testsuite this mostly leads to just +16 bytes code size increase (see table at D109817#3184752 <https://reviews.llvm.org/D109817#3184752>), so I believe it's acceptable fluctuation.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109817/new/
https://reviews.llvm.org/D109817
More information about the llvm-commits
mailing list