[all-commits] [llvm/llvm-project] e00cfe: [Local] Simplify the alignment limits in getOrEnfo...

topperc via All-commits all-commits at lists.llvm.org
Sat Apr 18 12:53:14 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: e00cfe254d99629ec344031adfe1878a84f3b0b3
      https://github.com/llvm/llvm-project/commit/e00cfe254d99629ec344031adfe1878a84f3b0b3
  Author: Craig Topper <craig.topper at gmail.com>
  Date:   2020-04-18 (Sat, 18 Apr 2020)

  Changed paths:
    M llvm/lib/Transforms/Utils/Local.cpp

  Log Message:
  -----------
  [Local] Simplify the alignment limits in getOrEnforceKnownAlignment. NFCI

We previously clamped the trailing zero count to 31 bits. And
then clamped the final alignment to MaximumAlignment which is
1 << 29.

This patch simplifies this to just clamp the trailing zero to
29 using MaxAlignmentExponent.

I was looking into changing this function to use Align/MaybeAlign
and noticed this.

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




More information about the All-commits mailing list