Ticket #7518 (closed task: fixed)
Instruction list length in bco->instrs redundant
|Reported by:||nomeata||Owned by:|
|Type of failure:||None/Unknown||Difficulty:||Unknown|
|Test Case:||Blocked By:|
While reading through the BCO-related code I notice that in the first word of the instruction list of a BCO (bco->instrs->payload), the length of the list (as the multiple of 16-bit-words) is stored. This is used in debugging code, e.g. in the disassembler.
However as far as I can tell this is duplicate information: bco->instrs->bytes already contains this information (as multiple of bytes).
Maybe the instruction list was not a proper StgArrWords? object before and this hack was required? If that is the case, I guess it can be removed now, saving a neglectable amount of memory and cleaning the code a bit.