[PATCH] D119994: [RewriteStatepointsForGC] Fix an incorrect assertion

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 17 02:37:02 PST 2022


skatkov added a comment.

The change itself looks good to me.



================
Comment at: llvm/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp:1180
           // llvm::Value of the correct type (and still remain pure).
           // This will remove the need to add bitcasts.
+          assert(StripBitCasts(Base) == StripBitCasts(OldBase) &&
----------------
Just wonder, don't you want to eliminate the reason for this assert?:)


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

https://reviews.llvm.org/D119994



More information about the llvm-commits mailing list