[PATCH] D107469: [LLVM][IR][NFC] Simplify writeDIGenericSubrange

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 13 09:31:19 PDT 2021


This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG17bc82dd3be9: [AsmWriter][NFC] Simplify writeDIGenericSubrange (authored by gAlfonso-bit, committed by MaskRay).

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107469/new/

https://reviews.llvm.org/D107469

Files:
  llvm/lib/IR/AsmWriter.cpp


Index: llvm/lib/IR/AsmWriter.cpp
===================================================================
--- llvm/lib/IR/AsmWriter.cpp
+++ llvm/lib/IR/AsmWriter.cpp
@@ -1931,10 +1931,9 @@
 
   auto IsConstant = [&](Metadata *Bound) -> bool {
     if (auto *BE = dyn_cast_or_null<DIExpression>(Bound)) {
-      return BE->isConstant()
-                 ? DIExpression::SignedOrUnsignedConstant::SignedConstant ==
-                       *BE->isConstant()
-                 : false;
+      return BE->isConstant() &&
+             DIExpression::SignedOrUnsignedConstant::SignedConstant ==
+                 *BE->isConstant();
     }
     return false;
   };


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D107469.366300.patch
Type: text/x-patch
Size: 652 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210813/1965a7e5/attachment.bin>


More information about the llvm-commits mailing list