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, 3 insertions, 3 deletions

diff --git a/source/slang/slang-ir-lower-bit-cast.cpp b/source/slang/slang-ir-lower-bit-cast.cpp
index ed312741e..fdb8a6bdb 100644
--- a/source/slang/slang-ir-lower-bit-cast.cpp
+++ b/source/slang/slang-ir-lower-bit-cast.cpp
@@ -30,7 +30,7 @@ struct BitCastLoweringContext
 
     void processInst(IRInst* inst)
     {
-        switch (inst->op)
+        switch (inst->getOp())
         {
         case kIROp_BitCast:
             processBitCast(inst);
@@ -70,7 +70,7 @@ struct BitCastLoweringContext
     // Extract an object of `type` from `offset` in `src`.
     IRInst* readObject(IRBuilder& builder, IRInst* src, IRType* type, uint32_t offset)
     {
-        switch (type->op)
+        switch (type->getOp())
         {
         case kIROp_StructType:
             {
@@ -187,7 +187,7 @@ struct BitCastLoweringContext
                         builder.getUInt64Type(),
                         high,
                         builder.getIntValue(builder.getUIntType(), 32)));
-                if (type->op == kIROp_UInt64Type)
+                if (type->getOp() == kIROp_UInt64Type)
                     return combined;
                 return builder.emitBitCast(type, combined);
             }