this fixes a bug where the instruction insert failed (for whatever reason), leaving tmp_errmsg `NULL', but later on in @arch@_cgen_assemble_insn(), it will pass this NULl tmp_errmsg to sprintf(). is this OK? would someone please commit it if so? It looks okay, but this isn't part of CGEN proper. Please send this patch to binutils@sources. Thanks, Ben