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-specialize-resources.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'source/slang/slang-ir-specialize-resources.cpp') diff --git a/source/slang/slang-ir-specialize-resources.cpp b/source/slang/slang-ir-specialize-resources.cpp index 9a51351a9..23e68182e 100644 --- a/source/slang/slang-ir-specialize-resources.cpp +++ b/source/slang/slang-ir-specialize-resources.cpp @@ -614,7 +614,7 @@ struct ResourceOutputSpecializationPass // At the very least, we expect them to be operations with // the same opcode. // - if(value->op != representative->op) + if(value->getOp() != representative->getOp()) return SLANG_FAIL; // Furthermore, only certain instructions are amenable to @@ -628,7 +628,7 @@ struct ResourceOutputSpecializationPass // Each supported instruction opcode might introduce new // constraints on how `value` and `representative` must match. // - switch( value->op ) + switch( value->getOp() ) { default: // Any opcode we do not specifically enable should cause @@ -1030,7 +1030,7 @@ struct ResourceOutputSpecializationPass // value in the context of the caller. // auto representative = info.representative; - switch( representative->op ) + switch( representative->getOp() ) { default: // Because we only allow certain instructions when specializing -- cgit v1.2.3