[Alignment] fix dubious min function alignment
authorGuillaume Chatelet <gchatelet@google.com>
Fri, 6 Sep 2019 13:54:09 +0000 (13:54 +0000)
committerGuillaume Chatelet <gchatelet@google.com>
Fri, 6 Sep 2019 13:54:09 +0000 (13:54 +0000)
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

llvm/lib/Target/SystemZ/SystemZISelLowering.cpp

index 5fd4252..d4d0905 100644 (file)
@@ -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);