[PATCH] D86864: [MachineSinking] sink more profitable loads
LiuChen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Dec 27 16:39:28 PST 2020
LiuChen3 added a comment.
> I can surely do that. But I think the most reasonable solution would be fix the compiling time issue. Since compiling time tests I did before does not expose any regression, your test case must be a little special. Could you find out the special point, for example the function has too many blocks or some/many blocks in the function has too many instructions? Thanks.
I think the increase in compile time is because the function has too many instructions and blocks. The function has Thousands of lines of instructions. Can you add limitation for the number of instructions or the number of blocks so the check for 'store' can end early?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D86864/new/
https://reviews.llvm.org/D86864
More information about the llvm-commits
mailing list