[all-commits] [llvm/llvm-project] f0f8b4: Recommit "[AggressiveInstCombine] Lower Table Base...

Djordje Todorovic via All-commits all-commits at lists.llvm.org
Tue Sep 20 04:13:57 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f0f8b4686392388e78b22339820f14a23b539937
      https://github.com/llvm/llvm-project/commit/f0f8b4686392388e78b22339820f14a23b539937
  Author: Djordje Todorovic <djordje.todorovic at syrmia.com>
  Date:   2022-09-20 (Tue, 20 Sep 2022)

  Changed paths:
    M llvm/lib/Transforms/AggressiveInstCombine/AggressiveInstCombine.cpp
    A llvm/test/Transforms/AggressiveInstCombine/lower-table-based-cttz-basics.ll
    A llvm/test/Transforms/AggressiveInstCombine/lower-table-based-cttz-dereferencing-pointer.ll
    A llvm/test/Transforms/AggressiveInstCombine/lower-table-based-cttz-non-argument-value.ll
    A llvm/test/Transforms/AggressiveInstCombine/lower-table-based-cttz-zero-element.ll
    A llvm/test/Transforms/AggressiveInstCombine/negative-lower-table-based-cttz.ll
    A llvm/test/Transforms/PhaseOrdering/lower-table-based-cttz.ll

  Log Message:
  -----------
  Recommit "[AggressiveInstCombine] Lower Table Based CTTZ

The bug reported on the [0] has been fixed.
The issue was we have not checked if the global variables that
represent cttz tables was constant.
There is a new negative test added in negative-lower-table-based-cttz.ll
that represents this.

[0] https://reviews.llvm.org/rGdf868edee561eb973edd85ec9df41c67aa0bff6b




More information about the All-commits mailing list