This is the mail archive of the mailing list for the binutils project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Ping: [PATCH] gas: fix bogus error on .org involving expression

>>> On 22.03.16 at 13:26, <> wrote:
> On Tue, 22 Mar 2016, Jan Beulich wrote:
>> Beyond those, however, this also revealed a few other failures:
>> 4) MMIX'es pr12815-* tests both fail (they no longer produce the
>> expected or any other error). Since I have no idea what exactly
>> those tests test, I also have no idea how to deal with this.
> You refer to *linker tests* checking for "a meaningful error
> message rather than SEGV" from the *linker*.  I suspect your
> assembler patches affects things it shouldn't, i.e. the it
> affects assembler output, not only fixing error cases.

While that's certainly true, it's entirely unclear to me why that would
be. Looking a little more closely, both generated object files have
empty .text and .data sections (yet only the .data section of the
second one was empty before). Assuming that what I see in the
sources are opcodes, I have absolutely no idea why that would
result in no code getting generated at all, but there also not being
any error. This, to me at least, quite clearly hints at a bug elsewhere
which this change uncovers. I could guess (and perhaps work out
by trial and error) whether one of the five uses of undefined_section
in gas/config/tc-mmix.c needs replacing or extending by considering
expr_section, but that's not the kind of work I'd like to do on code I
have no knowledge about at all.

Of course it's generally questionable whether the significantly
lower amount of expr_section uses in gas/config/tc-*.c compared
to undefined_section doesn't indicate further lurking issues. So
unless I can get some help here, I'm almost willing to give up on
this and leave .org broken (I've already managed to find an
acceptable replacement for the use case with which I ran into the
problem originally).


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]