Skip to content

[RISCV][P-ext] Remove duplicate hasSideEffects=0, mayLoad=0, mayStore=0. NFC#199798

Open
topperc wants to merge 1 commit into
llvm:mainfrom
topperc:pr/duplicate-let
Open

[RISCV][P-ext] Remove duplicate hasSideEffects=0, mayLoad=0, mayStore=0. NFC#199798
topperc wants to merge 1 commit into
llvm:mainfrom
topperc:pr/duplicate-let

Conversation

@topperc
Copy link
Copy Markdown
Contributor

@topperc topperc commented May 26, 2026

We had a let outside the class and inside.

…=0. NFC

We had a let outside the class and inside.
@llvmorg-github-actions
Copy link
Copy Markdown

@llvm/pr-subscribers-backend-risc-v

Author: Craig Topper (topperc)

Changes

We had a let outside the class and inside.


Full diff: https://github.com/llvm/llvm-project/pull/199798.diff

1 Files Affected:

  • (modified) llvm/lib/Target/RISCV/RISCVInstrInfoP.td (-4)
diff --git a/llvm/lib/Target/RISCV/RISCVInstrInfoP.td b/llvm/lib/Target/RISCV/RISCVInstrInfoP.td
index 37c33d27647bf..c0beb6b891284 100644
--- a/llvm/lib/Target/RISCV/RISCVInstrInfoP.td
+++ b/llvm/lib/Target/RISCV/RISCVInstrInfoP.td
@@ -284,10 +284,6 @@ class RVPWideningShift_ri<bits<3> f, string opcodestr, Operand ImmType>
   let Inst{11-8}  = rd{4-1};
   let Inst{7}     = 0b0;
   let Inst{6-0}   = OPC_OP_IMM_32.Value;
-
-  let hasSideEffects = 0;
-  let mayLoad = 0;
-  let mayStore = 0;
 }
 
 class RVPWideningShiftW_ri<bits<3> f, string opcodestr>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant