[PATCH] D147704: [Inliner] Preserve nonnull from callsites during inlining

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 24 09:48:05 PDT 2023


efriedma added a comment.
Herald added a subscriber: hoy.

Something "cheap" we could do is attach nonnull attributes to arguments of inlined calls, when appropriate.

llvm.assume is fundamentally pretty expensive; it bloats use-lists, and there's additional instructions feeding into the assume.  There are some aspects we could make cheaper with an optimized form of llvm.assume, but it's not easy.


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

https://reviews.llvm.org/D147704



More information about the llvm-commits mailing list