[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