[PATCH] D122148: [SLP] Peak into loads when hitting the RecursionMaxDepth

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 22 09:56:55 PDT 2022


dmgreen added a comment.

OK - it didn't increase the time very much either. Without the influence of D122145 <https://reviews.llvm.org/D122145>, these are the compile times I measured. The first column is this patch (almost no change). The second is increasing the limit from 12 to 14.

  			This (%)	14 limit (%)
  ClamAV			0.006357042	0.132623139
  7zip			0.01145899	0.003936277
  tramp3d-v4		-0.00551849	0.009901878
  kimwitu++		-0.000151601	0.003420248
  sqlite3			-0.005542751	0.002919453
  mafft			0.019282126	-0.006002062
  lencod			-0.0100786	0.029042137
  SPASS			0.006203808	0.014101154
  consumer-typeset	0.015334486	0.031552953
  Bullet			0.010231917	-0.017328356

Increasing the limit is still very small, but a little bigger in places than this patch (and may have worse performance in some degenerate cases).


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

https://reviews.llvm.org/D122148



More information about the llvm-commits mailing list