[all-commits] [llvm/llvm-project] 96345f: [IRSim] Remove early check from similarity matchin...

Andrew Litteken via All-commits all-commits at lists.llvm.org
Mon May 9 20:59:27 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 96345f773cfeb23b80aba8c9592446c661f1a0ef
      https://github.com/llvm/llvm-project/commit/96345f773cfeb23b80aba8c9592446c661f1a0ef
  Author: Andrew Litteken <andrew.litteken at gmail.com>
  Date:   2022-05-09 (Mon, 09 May 2022)

  Changed paths:
    M llvm/lib/Analysis/IRSimilarityIdentifier.cpp
    M llvm/unittests/Analysis/IRSimilarityIdentifierTest.cpp

  Log Message:
  -----------
  [IRSim] Remove early check from similarity matching such that commutative instructions are checked correctly when using the same value.

When the first commutative instruction in a region using the same value in both positions was compared to a corresponding instruction with two different values, there was an early check that determined that since the values were new, it was true that these values acted in the same way structurally. If this was not contradicted later in the program, the regions were marked as similar. This removes that check, so that it is clear that the same value cannot be mapped to two different values.

Reviewer: paquette

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




More information about the All-commits mailing list