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

1 files changed, 2 insertions, 2 deletions

diff --git a/source/slang/slang-ir-wrap-structured-buffers.cpp b/source/slang/slang-ir-wrap-structured-buffers.cpp
index 6b1bc5f2f..089f43625 100644
--- a/source/slang/slang-ir-wrap-structured-buffers.cpp
+++ b/source/slang/slang-ir-wrap-structured-buffers.cpp
@@ -113,7 +113,7 @@ struct WrapStructuredBuffersContext
         // replacing one type A with another type B globally, and doing
         // so could affect any type that in turn referenced A...
         //
-        auto newStructuredBufferType = builder->getType(oldStructuredBufferType->op, wrapperStruct);
+        auto newStructuredBufferType = builder->getType(oldStructuredBufferType->getOp(), wrapperStruct);
         oldStructuredBufferType->replaceUsesWith(newStructuredBufferType);
 
         // Any values that used our old structured bufer type
@@ -280,7 +280,7 @@ struct WrapStructuredBuffersContext
                         // non-pointer case above, so please refer
                         // there if you want the comments.
 
-                        auto newResultType = builder->getPtrType(oldPtrType->op, wrapperStruct);
+                        auto newResultType = builder->getPtrType(oldPtrType->getOp(), wrapperStruct);
                         builder->setDataType(call, newResultType);
 
                         auto newVal = builder->emitFieldAddress(oldResultType, call, wrappedFieldKey);