[PATCH] D119994: [RewriteStatepointsForGC] Fix an incorrect assertion
Daniil Suchkov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 17 10:17:22 PST 2022
DaniilSuchkov added inline comments.
================
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) &&
----------------
skatkov wrote:
> Just wonder, don't you want to eliminate the reason for this assert?:)
I guess we can just wait for opaque pointers, then this problem simply won't exist.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119994/new/
https://reviews.llvm.org/D119994
More information about the llvm-commits
mailing list