[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