switch qbrt to use READ_REG/WRITE_REG

these macros check for failures & promises so instruction code doesn't
have to
fix iffail instruction

it was getting caught up in framework and returning the failure not
branching on it. now there's framework to check the value even if it
is a failure