Author: Roman Lebedev
Date: 2020-01-21T21:18:29+03:00
New Revision: 6b2f820221c78c05527d6edb756b4f1f44665968

URL: 
https://github.com/llvm/llvm-project/commit/6b2f820221c78c05527d6edb756b4f1f44665968
DIFF: 
https://github.com/llvm/llvm-project/commit/6b2f820221c78c05527d6edb756b4f1f44665968.diff

LOG: [NFC][Codegen] Use MaybeAlign + APInt::getLimitedValue() when creating 
Alignment attr

Summary: Just an NFC code cleanup i stumbled upon when stumbling through clang 
alignment attribute handling.

Reviewers: erichkeane, gchatelet, courbet, jdoerfert

Reviewed By: gchatelet

Subscribers: cfe-commits

Tags: #clang

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

Added: 
    

Modified: 
    clang/lib/CodeGen/CGCall.cpp

Removed: 
    


################################################################################
diff  --git a/clang/lib/CodeGen/CGCall.cpp b/clang/lib/CodeGen/CGCall.cpp
index c5d73eb6ff22..69c0b0b03666 100644
--- a/clang/lib/CodeGen/CGCall.cpp
+++ b/clang/lib/CodeGen/CGCall.cpp
@@ -2440,9 +2440,8 @@ void CodeGenFunction::EmitFunctionProlog(const 
CGFunctionInfo &FI,
               EmitScalarExpr(AVAttr->getAlignment());
             llvm::ConstantInt *AlignmentCI =
               cast<llvm::ConstantInt>(AlignmentValue);
-            unsigned Alignment = 
std::min((unsigned)AlignmentCI->getZExtValue(),
-                                          +llvm::Value::MaximumAlignment);
-            AI->addAttrs(llvm::AttrBuilder().addAlignmentAttr(Alignment));
+            AI->addAttrs(llvm::AttrBuilder().addAlignmentAttr(llvm::MaybeAlign(
+                AlignmentCI->getLimitedValue(llvm::Value::MaximumAlignment))));
           }
         }
 


        
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to