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

--- Comment #5 from Christophe Lyon <clyon at gcc dot gnu.org> ---
(In reply to Nick Clifton from comment #4)
> (In reply to Christophe Lyon from comment #3)
> 
> Hi Christophe,
>  
> > It seems your patch does the trick, but I imagined we'd want a more general
> > solution (that is, do we want add such an optional prefix in all targets)?
> 
> Oh it is supported for all targets, but only when the default cpu scan()
> function is used.  (See bfd/archures.c:bfd_default_scan).  If the 
> architecture provides its own scan function, then this needs to support
> the <arch>:<machine> syntax in order for it to work.  (The AArch64 and ARM
> architectures being obvious culprits in this case).

Sorry, I wasn't aware of this feature, so that's a real bug in AArch64 and ARM
implementations.



> > That being said, doesn't adding such a prefix require updating some piece of
> > documentation?
> 
> Yes - you are right.  We should update the documentation as well.  Do you
> want
> to update the patch or shall I ?
> 
Please go ahead :-)

Thanks!

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

Reply via email to