public inbox for u-boot@lists.denx.de
 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox