[llvm] [SandboxIR] Add more Instruction member functions (PR #98588)
Arthur Eubanks via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 15 09:55:26 PDT 2024
================
@@ -262,6 +262,113 @@ const char *Instruction::getOpcodeName(Opcode Opc) {
llvm_unreachable("Unknown Opcode");
}
+llvm::Instruction *Instruction::getTopmostLLVMInstruction() const {
+ Instruction *Prev = getPrevNode();
+ if (Prev == nullptr) {
+ // If at top of the BB, return the first BB instruction.
+ return &*cast<llvm::BasicBlock>(getParent()->Val)->begin();
+ }
+ // Else get the Previous sandbox IR instruction's bottom IR instruction and
+ // return its successor.
+ llvm::Instruction *PrevBotI = cast<llvm::Instruction>(Prev->Val);
+ return PrevBotI->getNextNode();
+}
+
+BBIterator Instruction::getIterator() const {
+ auto *I = cast<llvm::Instruction>(Val);
+ return BasicBlock::iterator(I->getParent(), I->getIterator(), &Ctx);
+}
+
+Instruction *Instruction::getNextNode() const {
+ assert(getParent() != nullptr && "Detached!");
+ assert(getIterator() != getParent()->end() && "Already at end!");
+ auto *LLVMI = cast<llvm::Instruction>(Val);
+ assert(LLVMI->getParent() != nullptr && "LLVM IR instr is detached!");
+ auto *NextLLVMI = LLVMI->getNextNode();
----------------
aeubanks wrote:
I meant if `this` is a multi-instruction, then `NextLLVMI` will point to the second LLVM instruction in `this`. Or are you going to override `getNextNode()` in the multi-instruction subclass?
https://github.com/llvm/llvm-project/pull/98588
More information about the llvm-commits
mailing list