public inbox for cgen@sourceware.org
 help / color / mirror / Atom feed
From: Julius Baxter <juliusbaxter@gmail.com>
To: cgen@sourceware.org
Subject: Encoding of immediates different to label addresses
Date: Mon, 02 Jan 2012 09:44:00 -0000	[thread overview]
Message-ID: <CAF6YiZ7nkXDaviZB4r1FsHU+sC+95xMmToCO+Yaaf5qQ4WFgBw@mail.gmail.com> (raw)

Hi,

I'm working on fixing up the OpenRISC 1000 CGEN port and have come
across something I find I can't quite do with CGEN.

For the jump and branch instructions, if we use immediate value number
specified in the assembly, we need that encoded into the instruction
without any shifting occurring to it.

Eg. "l.bf 1" should become 0x10000001, "l.bf 4" should become 0x10000004 etc.

These target addresses are shifted-left by two during decoding, and so
"l.bf 1" is actually a branch of 4 bytes.

If we use a label for an address, though, I'm seeing that we're
getting the offset in bytes during encoding, which then needs to be
shifted right by two before being put in the instruction.

But it seems we can have the shifting on both numerical immediates and
labels, or neither. (Pardon my incorrect use of terminology, I have a
feeling I'm not quite calling everything its right name!)

Upon inspecttion, it looks like the binutils port we have been using
for the past few years (not CGEN-based) has a right-shift of 2 in the
md_apply_fix() function in binutils/gas/config/tc-or32.c (
http://sourceware.org/git/?p=binutils.git;a=blob;f=gas/config/tc-or32.c;h=7234fb837e006f979c255f62206e39a9822046d2;hb=HEAD#l603
) when the instruction had the actual value of the label inserted into
it.

Is there any way to handle this using the CGEN description?

The current instruction field definition being used is:

(df f-disp26     "disp26"              (PCREL-ADDR) 25 26 INT
    ((value pc) (sra WI (sub WI value pc) (const 2)))
    ((value pc) (add WI (sll WI value (const 2)) pc)))

This does the shifting on encode and the target label addresses are
right but the numerical immediates are wrong, but if I remove the sra
on the encode, the numerical immediates are right but the target label
addresses are wrong.

I've had a bit of a poke around and seen the function
gas_cgen_md_apply_fix() is used as a generic md_apply_fix() function,
but can't quite see where I might be able to hook in things for
particular encoding cases.

If anyone could point out how I could handle this case it'd be appreciated.

Thanks,

Julius

             reply	other threads:[~2012-01-02  9:44 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-02  9:44 Julius Baxter [this message]
2012-01-03 10:57 ` Andrew Burgess

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAF6YiZ7nkXDaviZB4r1FsHU+sC+95xMmToCO+Yaaf5qQ4WFgBw@mail.gmail.com \
    --to=juliusbaxter@gmail.com \
    --cc=cgen@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).