From: Guillaume Chatelet <gchatelet@google.com>
Date: Fri, 6 Sep 2019 13:54:09 +0000 (+0000)
Subject: [Alignment] fix dubious min function alignment
X-Git-Tag: llvmorg-11-init~9843
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5d870c2ec0a631f41ec4387e9d3c6073fea9c178;p=platform%2Fupstream%2Fllvm.git

[Alignment] fix dubious min function alignment

Summary:
This was discovered while introducing the llvm::Align type.
The original setMinFunctionAlignment used to take alignment as log2, looking at the comment it seems like instructions are to be 2-bytes aligned and not 4-bytes aligned.

Reviewers: uweigand

Subscribers: hiraditya, llvm-commits

Tags: #llvm

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

llvm-svn: 371204
---

diff --git a/llvm/lib/Target/SystemZ/SystemZISelLowering.cpp b/llvm/lib/Target/SystemZ/SystemZISelLowering.cpp
index 5fd4252..d4d0905 100644
--- a/llvm/lib/Target/SystemZ/SystemZISelLowering.cpp
+++ b/llvm/lib/Target/SystemZ/SystemZISelLowering.cpp
@@ -120,7 +120,7 @@ SystemZTargetLowering::SystemZTargetLowering(const TargetMachine &TM,
   setBooleanVectorContents(ZeroOrNegativeOneBooleanContent);
 
   // Instructions are strings of 2-byte aligned 2-byte values.
-  setMinFunctionAlignment(llvm::Align(4));
+  setMinFunctionAlignment(llvm::Align(2));
   // For performance reasons we prefer 16-byte alignment.
   setPrefFunctionLogAlignment(4);