[PATCH] D137707: [WIP] Move "auto-init" instructions to the dominator of their users

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 17 14:11:52 PST 2022


efriedma added a comment.

> I'd appreciate pointer to the literature on "sinking pass".

Look for PRE (partial redundancy elimination) of stores or partial dead store elimination.  Maybe start with https://dl.acm.org/doi/10.1145/277650.277659 .  See also D29865 <https://reviews.llvm.org/D29865> .

> So it appears that this patch made no difference to the size of the .text section.

This isn't fundamentally surprising; you have the same number of stores before and after sinking.


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

https://reviews.llvm.org/D137707



More information about the llvm-commits mailing list