[all-commits] [llvm/llvm-project] 360759: [SCEV] Add test for umin_seq with duplicate operands

Roman Lebedev via All-commits all-commits at lists.llvm.org
Tue Jan 11 05:52:14 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 36075942f99825f72854a53b045594497c3dba3d
      https://github.com/llvm/llvm-project/commit/36075942f99825f72854a53b045594497c3dba3d
  Author: Roman Lebedev <lebedev.ri at gmail.com>
  Date:   2022-01-11 (Tue, 11 Jan 2022)

  Changed paths:
    M llvm/test/Analysis/ScalarEvolution/exit-count-select-safe.ll

  Log Message:
  -----------
  [SCEV] Add test for umin_seq with duplicate operands


  Commit: 5e166507920165dfb08bf3a57aec80776929fcad
      https://github.com/llvm/llvm-project/commit/5e166507920165dfb08bf3a57aec80776929fcad
  Author: Roman Lebedev <lebedev.ri at gmail.com>
  Date:   2022-01-11 (Tue, 11 Jan 2022)

  Changed paths:
    M llvm/lib/Analysis/ScalarEvolution.cpp
    M llvm/test/Analysis/ScalarEvolution/exit-count-select-safe.ll

  Log Message:
  -----------
  [SCEV] `getSequentialMinMaxExpr()`: keep only the first instance of an operand

Having the same operand more than once doesn't change the outcome here,
neither reduction-wise nor poison-wise.
We must keep the first instance specifically though.


Compare: https://github.com/llvm/llvm-project/compare/0eef65028e8a...5e1665079201


More information about the All-commits mailing list