[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