[PATCH] D94482: [Verifier] Extend statepoint verifier to cover more constants
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 12 09:03:51 PST 2021
reames added a comment.
LGTM
Side note: The -1/+1s appearing everywhere confused me until I looked into the getConstMetaVal function. Having the index returned be the second in the tag, value pair seems slightly confusing, but it fits the existing structure.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94482/new/
https://reviews.llvm.org/D94482
More information about the llvm-commits
mailing list