[PATCH] D75598: [InstCombine] Handle gc.relocate(null) in one iteration

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 13 07:44:59 PDT 2020


reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.

LGTM

Serguei and I discussed this one fairly extensively offline.  Long term, we're probably going to move to a model where we process all of the projections for a statepoint in a single pass when visiting the statepoint, but for the moment, we need to decrease the number of instcombine iterations.  In particular, with recent efforts to clamp the number of iterations in debug builds, we are seeing R+A crashes in downstream tests, so this change is really a regression fix.


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

https://reviews.llvm.org/D75598



More information about the llvm-commits mailing list