bug-binutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Bug gas/30856] Regression: operand size mismatch for `push'


From: jbeulich at suse dot com
Subject: [Bug gas/30856] Regression: operand size mismatch for `push'
Date: Thu, 21 Sep 2023 13:01:21 +0000

https://sourceware.org/bugzilla/show_bug.cgi?id=30856

Jan Beulich <jbeulich at suse dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jbeulich at suse dot com
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |ASSIGNED
           Assignee|unassigned at sourceware dot org   |jbeulich at suse dot com
   Last reconfirmed|                            |2023-09-21

--- Comment #3 from Jan Beulich <jbeulich at suse dot com> ---
I'll look into why this happens, but:

1) I'm not sure we do ourselves (and our users) much good if we allow this
again. The referenced symbol is severely constrained in where in the address
space it may live. Nevertheless similar symbol uses (with e.g. MOV or ADD)
still work, so it is likely necessary to fix this at least for consistency
reasons, and for --x32.

2) I'm not convinced the compiler should actually emit such code. It can't know
where the linker script (which may be a custom one) would put the executable.
When put outside of the low 2G of address space and outside of the high 2G of
address space, linking would fail from what I can tell. The emitted code looks
to be legitimate only when -mcmodel=kernel is in effect, or in --x32 mode.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]