[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