[llvm] PeepholeOpt: Remove null TargetRegisterInfo check (PR #123933)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 22 17:51:04 PST 2025


https://github.com/arsenm updated https://github.com/llvm/llvm-project/pull/123933

>From 32796a97435de9140b66706e016bf7b805778176 Mon Sep 17 00:00:00 2001
From: Matt Arsenault <Matthew.Arsenault at amd.com>
Date: Wed, 22 Jan 2025 18:51:26 +0700
Subject: [PATCH] PeepholeOpt: Remove null TargetRegisterInfo check

This cannot happen. Also simplify the LaneBitmask check from !none
to any.
---
 llvm/lib/CodeGen/PeepholeOptimizer.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/llvm/lib/CodeGen/PeepholeOptimizer.cpp b/llvm/lib/CodeGen/PeepholeOptimizer.cpp
index d56f040cf421fd..5fc8f419e80a5d 100644
--- a/llvm/lib/CodeGen/PeepholeOptimizer.cpp
+++ b/llvm/lib/CodeGen/PeepholeOptimizer.cpp
@@ -2048,9 +2048,9 @@ ValueTrackerResult ValueTracker::getNextSourceFromInsertSubreg() {
   // Get the TRI and check if the inserted sub-register overlaps with the
   // sub-register we are tracking.
   const TargetRegisterInfo *TRI = MRI.getTargetRegisterInfo();
-  if (!TRI || !(TRI->getSubRegIndexLaneMask(DefSubReg) &
-                TRI->getSubRegIndexLaneMask(InsertedReg.SubIdx))
-                   .none())
+  if ((TRI->getSubRegIndexLaneMask(DefSubReg) &
+       TRI->getSubRegIndexLaneMask(InsertedReg.SubIdx))
+          .any())
     return ValueTrackerResult();
   // At this point, the value is available in v0 via the same subreg
   // we used for Def.



More information about the llvm-commits mailing list