Use TLMessages constants instead of numbers

This commit is contained in:
Hansung Kim
2023-04-18 00:09:48 -07:00
parent 2ac5ee398a
commit 65a22b7fcb

View File

@@ -801,16 +801,18 @@ class MemTraceLogger(
) )
def tlAOpcodeIsStore(opcode: UInt): Bool = { def tlAOpcodeIsStore(opcode: UInt): Bool = {
// 0: PutFullData, 1: PutPartialData but we don't support it assert(
// 4: Get opcode === TLMessages.PutFullData || opcode === TLMessages.PutPartialData,
assert(opcode === 0.U || opcode === 4.U, "unhandled TL A opcode found") "unhandled TL A opcode found"
opcode === 0.U )
opcode === TLMessages.PutFullData
} }
def tlDOpcodeIsStore(opcode: UInt): Bool = { def tlDOpcodeIsStore(opcode: UInt): Bool = {
// 0: AccessAck (Put), 1: AccessAckData (Get or Atomic) assert(
// See Table 13 of spec 1.8.1 opcode === TLMessages.AccessAck || opcode === TLMessages.AccessAckData,
assert(opcode === 0.U || opcode === 1.U, "unhandled TL D opcode found") "unhandled TL D opcode found"
opcode === 0.U )
opcode === TLMessages.AccessAck
} }
// snoop on the TileLink edges to log traffic // snoop on the TileLink edges to log traffic