[PATCH] D19421: [RewriteStatepointsForGC] Stabilise rematerialization order

Igor Laevsky via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 25 09:43:54 PDT 2016


igor-laevsky added a comment.

Hi Sanjoy,

It would be preferable to have determinism by construction. However in order to
get it we will need to not only change StatepointLiveSetTy type, but also
change all data types associated with liveness/basepointer analysis.

I'm afraid by doing so we will spend considerable amount of resources
(SetVector's and MapVector's are more expensive). And since this issue is not
directly related to correctness I would go with this non-perfect approach.


http://reviews.llvm.org/D19421





More information about the llvm-commits mailing list