[all-commits] [llvm/llvm-project] 3667d8: Double check that passes correctly set their Modif...

serge-sans-paille via All-commits all-commits at lists.llvm.org
Tue Jul 14 01:00:48 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 3667d87a33d3c8d4072a41fd84bb880c59347dc0
      https://github.com/llvm/llvm-project/commit/3667d87a33d3c8d4072a41fd84bb880c59347dc0
  Author: serge-sans-paille <sguelton at redhat.com>
  Date:   2020-07-14 (Tue, 14 Jul 2020)

  Changed paths:
    M llvm/lib/IR/LegacyPassManager.cpp
    M llvm/unittests/IR/LegacyPassManagerTest.cpp

  Log Message:
  -----------
  Double check that passes correctly set their Modified status

The approach is simple: if a pass reports that it's not modifying a
Function/Module, compute a loose hash of that Function/Module and compare it
with the original one. If we report no change but there's a hash change, then we
have an error.

This approach misses a lot of change but it's not super intrusive and can
detect most of the simple mistakes.

Differential Revision: https://reviews.llvm.org/D80916




More information about the All-commits mailing list