[PATCH] D41463: [CodeGen] Add a new pass for PostRA sink

Jun Bum Lim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 23 09:33:59 PDT 2018


junbuml added a comment.

> This pass destroys DominatorInfo and we have to recompute it right after the pass from scratch. Is it possible to preserve it? Also, have you measured compile time impact of the patch?

I measured compile time impact of this patch for spec2000/2006/2017. Overall, I wasn't able to see any reproduciable regression; all up and down are in noise range. There is no change in CFG in this pass, preserve DT should be good and I will submit a follow-up patch for it.


Repository:
  rL LLVM

https://reviews.llvm.org/D41463





More information about the llvm-commits mailing list