All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bill Pringlemeir <bpringlemeir@nbsps.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] Query on CONFIG_SYS_THUMB_BUILD
Date: Thu, 20 Nov 2014 11:34:45 -0500	[thread overview]
Message-ID: <87zjblltju.fsf@nbsps.com> (raw)
In-Reply-To: 20141120130425.182928c3@lilith


> On Wed, 19 Nov 2014 13:34:34 -0500, Bill Pringlemeir
> <bpringlemeir@nbsps.com> wrote:

>>>>> In message <20141119074214.3d414ce6@lilith> Albert wrote:

>>>>>> For -mauto-it, it is not documented in the gas documentation
>>>>>> online or in my current as' --target-help. I'll dig this deeper
>>>>>> today, but barring any scream from me, the change above is fine
>>>>>> globally in U-Boot.

>>>> On 19 Nov 2014, wd at denx.de wrote:
>>
>>>>> Apparently this [1] is where it is coming from; no further
>>>>> documentation there, though.
>>>>
>>>>> [1] https://sourceware.org/ml/binutils/2009-05/msg00132.html
>>
>>> On Wed, 19 Nov 2014 11:31:05 -0500, Bill Pringlemeir
>>
>>>> I would think that if this worked they would make it automatic and
>>>> not an option.  Probably this is only in certain binutils/as.

>>>> With 4.6.3 and 4.9.1 I do not have this option,

>> On 19 Nov 2014, albert.u.boot at aribaud.net wrote:

>>> Which option do you mean? -mimplicit-it or -mauto-it?

>> '-mauto-it' , which I think if it is working correctly would be
>> rolled into '-mimplicit-it' as it generates better code (for an
>> assembler :).  I followed the thread above and the patch originator
>> says he needs to fix section issues and the 'command line options'
>> and he would follow up the proposed patch.

On 20 Nov 2014, albert.u.boot at aribaud.net wrote:

> I am getting lost, even when reading (quickly, I admit) the patch that
> adds it; I don't see what -mauto-it does exactly. Can you summarize
> and clarify the effects of -mimplicit-it (I guess I know this one but
> it's never a bad thing to get a second opinion), -mauto-it and their
> interaction?

I guess you know how the 'IT' works.  The Ubuntu/Debian people give a
good explanation in a few paragraphs,

 https://wiki.ubuntu.com/ARM/Thumb2PortingHowto#Conditional_Execution

My trying to explain this may have confused thing...

Here is Wolfgang's reference,

 https://sourceware.org/ml/binutils/2009-05/msg00132.html

Here is a 2nd reference,

 https://sourceware.org/ml/binutils/2009-06/msg00162.html

Originally Daniel Gutson used '-mauto-it' and then it was converted to
'-mimplicit-it'.

I am not sure if '-mauto-it' exists in the wild.  I have never heard of
that option before seeing this email thread. Also my assembler says,

   Assembler messages:
   Error: unrecognized option -mauto-it

I have built with the most recent binutils, gcc4.9.1 using crosstool-ng.
Maybe only some non-mainline tools picked up this '-mauto-it' patch.  I
don't think it hurts to support '-mauto-it', but an assembler test
should be done to see if it accepts the option.

hth,
Bill Pringlemeir.

  reply	other threads:[~2014-11-20 16:34 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-13 16:29 [U-Boot] Query on CONFIG_SYS_THUMB_BUILD Victor Ascroft
2014-11-13 20:53 ` Wolfgang Denk
2014-11-14  4:30   ` Victor Ascroft
2014-11-14  5:43     ` Wolfgang Denk
2014-11-14  6:10       ` Victor Ascroft
2014-11-14 14:01 ` Simon Glass
2014-11-14 15:26   ` Albert ARIBAUD
2014-11-15  1:56     ` Simon Glass
2014-11-15  5:26       ` Victor Ascroft
2014-11-15  5:38         ` Victor Ascroft
2014-11-15 12:30       ` Albert ARIBAUD
2014-11-15 22:10         ` Simon Glass
2014-11-16  7:50           ` Albert ARIBAUD
2014-11-17  6:28             ` Simon Glass
2014-11-18  3:32               ` Victor Ascroft
2014-11-18  4:59                 ` Simon Glass
2014-11-18 12:10   ` Stefan Agner
2014-11-18 16:07   ` Stefan Agner
2014-11-18 18:37     ` Stefan Agner
2014-11-19  6:42       ` Albert ARIBAUD
2014-11-19  6:58         ` Wolfgang Denk
2014-11-19 16:31           ` Bill Pringlemeir
2014-11-19 17:48             ` Albert ARIBAUD
2014-11-19 18:34               ` Bill Pringlemeir
2014-11-20 12:04                 ` Albert ARIBAUD
2014-11-20 16:34                   ` Bill Pringlemeir [this message]
2014-11-21 12:22                     ` Albert ARIBAUD

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=87zjblltju.fsf@nbsps.com \
    --to=bpringlemeir@nbsps.com \
    --cc=u-boot@lists.denx.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.