[all-commits] [llvm/llvm-project] 961483: [NFCI][Local] Rewrite EliminateDuplicatePHINodes t...

Roman Lebedev via All-commits all-commits at lists.llvm.org
Sat Aug 29 12:04:25 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 961483a5ea7c0e628c025187287d1658457ffcb3
      https://github.com/llvm/llvm-project/commit/961483a5ea7c0e628c025187287d1658457ffcb3
  Author: Roman Lebedev <lebedev.ri at gmail.com>
  Date:   2020-08-29 (Sat, 29 Aug 2020)

  Changed paths:
    M llvm/lib/Transforms/Utils/Local.cpp

  Log Message:
  -----------
  [NFCI][Local] Rewrite EliminateDuplicatePHINodes to optionally check hashing invariants

EarlyCSE has a mode to verify the invariant that hash equality equals
key equality, but EliminateDuplicatePHINodes() doesn't.

I've verified that this would have caught the stage2-stage3 mismatches
5ec2b757cc7d37ff0d03b36ee863b0962fe78108 revert has fixed,
that were introduced last time in 3e69871ab5a66fb55913a2a2f5e7f5b42899a4c9.


  Commit: 1dcb936cf693b7f7abd271edc27df13425ebaac0
      https://github.com/llvm/llvm-project/commit/1dcb936cf693b7f7abd271edc27df13425ebaac0
  Author: Roman Lebedev <lebedev.ri at gmail.com>
  Date:   2020-08-29 (Sat, 29 Aug 2020)

  Changed paths:
    M llvm/lib/Transforms/Utils/Local.cpp

  Log Message:
  -----------
  [NFC][Local] EliminateDuplicatePHINodes(): add STATISTIC()


Compare: https://github.com/llvm/llvm-project/compare/1364d856f4bf...1dcb936cf693


More information about the All-commits mailing list