[llvm] 167da6c - [AArch64][GlobalISel] Clarify fallback debug print

Jon Roelofs via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 20 12:41:24 PDT 2021


Author: Jon Roelofs
Date: 2021-04-20T12:41:14-07:00
New Revision: 167da6c9e826b9befc27b6f5a6c540f7f28b7d5a

URL: https://github.com/llvm/llvm-project/commit/167da6c9e826b9befc27b6f5a6c540f7f28b7d5a
DIFF: https://github.com/llvm/llvm-project/commit/167da6c9e826b9befc27b6f5a6c540f7f28b7d5a.diff

LOG: [AArch64][GlobalISel] Clarify fallback debug print

... to only print when that fallback actually happens.

Added: 
    

Modified: 
    llvm/lib/Target/AArch64/GISel/AArch64CallLowering.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/AArch64/GISel/AArch64CallLowering.cpp b/llvm/lib/Target/AArch64/GISel/AArch64CallLowering.cpp
index 17dca37be12b..7d8b951e38a5 100644
--- a/llvm/lib/Target/AArch64/GISel/AArch64CallLowering.cpp
+++ b/llvm/lib/Target/AArch64/GISel/AArch64CallLowering.cpp
@@ -450,9 +450,10 @@ bool AArch64CallLowering::fallBackToDAGISel(const MachineFunction &MF) const {
       }))
     return true;
   const auto &ST = MF.getSubtarget<AArch64Subtarget>();
-  LLVM_DEBUG(dbgs() << "Falling back to SDAG because we don't support no-NEON");
-  if (!ST.hasNEON() || !ST.hasFPARMv8())
+  if (!ST.hasNEON() || !ST.hasFPARMv8()) {
+    LLVM_DEBUG(dbgs() << "Falling back to SDAG because we don't support no-NEON\n");
     return true;
+  }
   return false;
 }
 


        


More information about the llvm-commits mailing list