[all-commits] [llvm/llvm-project] 34fe8b: [test][AggressiveInstCombine] Precommit testcase f...

mikaelholmen via All-commits all-commits at lists.llvm.org
Thu Oct 26 01:00:28 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 34fe8be705c608c4c25ece725eef9305ccfa64e1
      https://github.com/llvm/llvm-project/commit/34fe8be705c608c4c25ece725eef9305ccfa64e1
  Author: Mikael Holmen <mikael.holmen at ericsson.com>
  Date:   2023-10-26 (Thu, 26 Oct 2023)

  Changed paths:
    A llvm/test/Transforms/AggressiveInstCombine/AArch64/combine_ignore_debug.ll

  Log Message:
  -----------
  [test][AggressiveInstCombine] Precommit testcase for #69925

We get different results with/without debug info present.


  Commit: ce0a750fe4b04630fcb242822627d790f4c2878b
      https://github.com/llvm/llvm-project/commit/ce0a750fe4b04630fcb242822627d790f4c2878b
  Author: Mikael Holmen <mikael.holmen at ericsson.com>
  Date:   2023-10-26 (Thu, 26 Oct 2023)

  Changed paths:
    M llvm/lib/Transforms/AggressiveInstCombine/AggressiveInstCombine.cpp
    M llvm/test/Transforms/AggressiveInstCombine/AArch64/combine_ignore_debug.ll

  Log Message:
  -----------
  [AggressiveInstCombine] Ignore debug instructions when load combining (#70200)

We previously included debug instructions when counting instructions when
looking for loads to combine. This meant that the presence of debug
instructions could affect optimization, as shown in the updated testcase.

This fixes #69925.


Compare: https://github.com/llvm/llvm-project/compare/e58c4c771329...ce0a750fe4b0


More information about the All-commits mailing list