From e474c4e3aadc22a1b9f9b006104409f10936244f Mon Sep 17 00:00:00 2001 From: Tim Foley Date: Tue, 16 Feb 2021 11:48:21 -0800 Subject: Add an accessor for IRInst opcode (#1707) * Add an accessor for IRInst opcode This main changing is renaming `IRInst::op` over to `IRInst::m_op` and then adds an accessor `IRInst::getOp()` to read it. The rest of the changes are just changing use sites to `getOp` (or to `m_op` in the limited cases where we write to it). This work is in anticipation of a future change that might need to store an extra bit in the same field as the opcode. It seemed better to do this massive refactoring as a separate PR. * fixup --- source/slang/slang-ir-restructure-scoping.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'source/slang/slang-ir-restructure-scoping.cpp') diff --git a/source/slang/slang-ir-restructure-scoping.cpp b/source/slang/slang-ir-restructure-scoping.cpp index c16db8f3c..0b8e37d31 100644 --- a/source/slang/slang-ir-restructure-scoping.cpp +++ b/source/slang/slang-ir-restructure-scoping.cpp @@ -150,7 +150,7 @@ IRInst* getDefaultInitVal( IRBuilder* builder, IRType* type) { - switch( type->op ) + switch( type->getOp() ) { default: return nullptr; @@ -181,7 +181,7 @@ void defaultInitializeVar( IRType* type) { IRInst* initVal = nullptr; - switch( type->op ) + switch( type->getOp() ) { case kIROp_VoidType: default: @@ -217,7 +217,7 @@ static void fixValueScopingForInst( // because the emit logic will already create variables for them. // We could consider folding the logic to move out of SSA form // into this function, but that would add a lot of complexity for now. - if(def->op == kIROp_Param) + if(def->getOp() == kIROp_Param) return; // We would have a scoping violation if there exists some -- cgit v1.2.3