[llvm] fb8ecca - [RISCV] Remove redundant code checking for exact VTYPE match [nfc]

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 17 12:21:30 PDT 2022


Author: Philip Reames
Date: 2022-06-17T12:20:20-07:00
New Revision: fb8ecca06f6a9798f0e246853749c68fa17e18e9

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

LOG: [RISCV] Remove redundant code checking for exact VTYPE match [nfc]

Should be fully covered by the generic demanded field based logic just below, and this ensures better coverage of that logic.

Added: 
    

Modified: 
    llvm/lib/Target/RISCV/RISCVInsertVSETVLI.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/RISCV/RISCVInsertVSETVLI.cpp b/llvm/lib/Target/RISCV/RISCVInsertVSETVLI.cpp
index 321034658335..5b84bb4e5e25 100644
--- a/llvm/lib/Target/RISCV/RISCVInsertVSETVLI.cpp
+++ b/llvm/lib/Target/RISCV/RISCVInsertVSETVLI.cpp
@@ -603,10 +603,6 @@ class VSETVLIInfo {
 
   bool hasCompatibleVTYPE(const MachineInstr &MI,
                           const VSETVLIInfo &Require) const {
-    // Simple case, see if full VTYPE matches.
-    if (hasSameVTYPE(Require))
-      return true;
-
     const DemandedFields Used = getDemanded(MI);
     return areCompatibleVTYPEs(encodeVTYPE(), Require.encodeVTYPE(), Used);
   }


        


More information about the llvm-commits mailing list