[PATCH] D80916: [LegacyPM] Double check that passes correctly set their Modified status

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 3 08:46:33 PDT 2020


foad added a comment.

I like this and it is certainly useful for finding bugs. I'd like to see it for Loop, Region and SCC passes too. But I don't know if it's fast and/or robust enough to commit and have it enabled as-is.

Is there any other code for hashing Functions and Modules that we could use here instead of rolling our own?


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

https://reviews.llvm.org/D80916





More information about the llvm-commits mailing list