Re: Fix for parallel_ok in d10v assembler

[Matthew Hiller <hiller at redhat dot com>]

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