Fix for parallel_ok in d10v assembler
Matthew Hiller
hiller@redhat.com
Mon Dec 4 17:27:00 GMT 2000
On Mon, 4 Dec 2000, Nick Clifton wrote:
> This does not look right. If the brf0t.s instruction cannot be
> parallelized then it should have the ALONE attribute set (in
> opcodes/d10v-opc.c). That is the point of that attribute. It should
> not be necessary to add tests for the BRANCH attribute in parallel_ok.
Not precisely. It's perfectly okay for brf0t.s to be in a
sequenced packed instruction, which ALONE doesn't allow for. e.g., in the
testcase that I'm working with,
brf0t.s 14 <func_a+0x4> -> mv r0, r2
It's just not okay for it to appear in a in a parallel packed
instruction, e.g.,
brf0t.s 14 <func_a+0x4> || mv r0, r2
Without the check for the BRANCH attribute, when the assembler is
run with -O, it will wrongly decide that the sequence can be put in a
parallel packed instruction rather than a sequential.
Matt
--
Matt Hiller
GCC Engineer, Red Hat, Inc., Sunnyvale office
hiller@redhat.com
Phone: (408) 542-9643 Fax: (408) 542-9743
More information about the Binutils
mailing list