[llvm] 8bdb149 - DAG: Remove redundant check for return alignment

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 31 04:44:09 PST 2023


Author: Matt Arsenault
Date: 2023-01-31T08:43:56-04:00
New Revision: 8bdb149c0a544611c06cbdfd6cd5151ca1589dcb

URL: https://github.com/llvm/llvm-project/commit/8bdb149c0a544611c06cbdfd6cd5151ca1589dcb
DIFF: https://github.com/llvm/llvm-project/commit/8bdb149c0a544611c06cbdfd6cd5151ca1589dcb.diff

LOG: DAG: Remove redundant check for return alignment

This is already what the CallBase getRetAlign does

Added: 
    

Modified: 
    llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
index 2c96b8f3068d..dc51bf517246 100644
--- a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
+++ b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
@@ -4918,8 +4918,7 @@ void SelectionDAGBuilder::visitTargetIntrinsic(const CallInst &I,
       Result = lowerRangeToAssertZExt(DAG, I, Result);
 
     MaybeAlign Alignment = I.getRetAlign();
-    if (!Alignment)
-      Alignment = F->getAttributes().getRetAlignment();
+
     // Insert `assertalign` node if there's an alignment.
     if (InsertAssertAlign && Alignment) {
       Result =


        


More information about the llvm-commits mailing list