[all-commits] [llvm/llvm-project] 91722a: [LoopRotate] Don't rotate loops when the minsize a...

Ayke via All-commits all-commits at lists.llvm.org
Tue Jul 16 07:10:18 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 91722a4a13d808772edcdd5caad66a598a659655
      https://github.com/llvm/llvm-project/commit/91722a4a13d808772edcdd5caad66a598a659655
  Author: Ayke <aykevanlaethem at gmail.com>
  Date:   2024-07-16 (Tue, 16 Jul 2024)

  Changed paths:
    M llvm/lib/Transforms/Scalar/LoopRotation.cpp
    A llvm/test/Transforms/LoopRotate/minsize-disable.ll

  Log Message:
  -----------
  [LoopRotate] Don't rotate loops when the minsize attribute is present

The main use for this patch is LTO. It is not (yet?) possible to set the
size level (-Os, -Oz) in the linker, which means loops are still rotated
even if -Oz is specified on the command line. Therefore, look at the
function attribute instead of only at the size level to determine
whether to rotate loops for a given function.

For discussion, see: https://reviews.llvm.org/D119342

An older version of this patch was already approved at
https://reviews.llvm.org/D119342 but I never got around to committing
it. The code changed so I had to make some minor updates to this patch
and in the meantime I also lost commit access because I wasn't really
using it. So here is an updated patch.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list