From 7fe45ec6538c1b9675548181d4a589eafbe0beb7 Mon Sep 17 00:00:00 2001 From: Guillaume Chatelet Date: Fri, 6 Sep 2019 13:54:09 +0000 Subject: [PATCH] [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 git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@371204 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Target/SystemZ/SystemZISelLowering.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/Target/SystemZ/SystemZISelLowering.cpp b/lib/Target/SystemZ/SystemZISelLowering.cpp index 5fd4252599c..d4d0905545c 100644 --- a/lib/Target/SystemZ/SystemZISelLowering.cpp +++ b/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); -- 2.11.4.GIT