[PATCH] D141153: [LocalStackSlotAllocation] Minor simplifications. NFC

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 9 04:58:43 PST 2023


arsenm accepted this revision.
arsenm added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/CodeGen/LocalStackSlotAllocation.cpp:388
     // register.
-    if (UsedBaseReg &&
+    if (BaseReg.isValid() &&
         lookupCandidateBaseReg(BaseReg, BaseOffset, FrameSizeAdjust,
----------------
Conversion to bool also works, you don't need an explicit isValid


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141153



More information about the llvm-commits mailing list