]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
[committed] Fix HImode load mnemonic on microblaze port
authorJeff Law <jlaw@ventanamicro.com>
Mon, 4 Dec 2023 17:06:49 +0000 (10:06 -0700)
committerJeff Law <jlaw@ventanamicro.com>
Mon, 4 Dec 2023 17:06:49 +0000 (10:06 -0700)
The tester recently started failing va-arg-22.c on microblaze-linux:

gcc.c-torture/execute/va-arg-22.c   -O0  (test for excess errors)

It was failing with an undefined reference to "r7" at link time.  This was
ultimately tracked down to a HImode load using (reg+reg) addressing mode, but
which used the lhui instruction instead of lhu.  The "i" means it's supposed to
be (reg+disp) so the assembler tried to interpret "r7" as an immediate/symbol.

The port uses %i<opnum> as an output modifier to select between sh/shi and
various other mnemonics for loads/stores.  The movhi pattern simply failed to
use it for the two cases where it's loading from memory (interestingly enough
it was used for stores).

Clearly we aren't using reg+reg much for HImode loads as this didn't fix
anything else in the testsuite.

gcc/
* config/microblaze/microblaze.md (movhi): Use %i for half-word
loads to properly select between lhu/lhui.

gcc/config/microblaze/microblaze.md

index 671667b537cb3fe76568d4701b7d13599b9a3abe..a8ee886d36b0c216ff375276b904aae7b1fa6f30 100644 (file)
   "@
    addik\t%0,r0,%1\t# %X1
    addk\t%0,%1,r0
-   lhui\t%0,%1
-   lhui\t%0,%1
+   lhu%i1\t%0,%1
+   lhu%i1\t%0,%1
    sh%i0\t%z1,%0
    sh%i0\t%z1,%0"
   [(set_attr "type"    "arith,move,load,no_delay_load,store,no_delay_store")