https://sourceware.org/bugzilla/show_bug.cgi?id=20030
--- Comment #3 from Christophe Monat <christophe.monat at st dot com> ---
(In reply to Nick Clifton from comment #1)

Hi Nick,
>   Will something like this do ?

Thanks for this, it looks good, but I am seeing that the DP registers are
grouped by pairs whereas they could be group by four, meaning that we could
keep only one veneer size for VLDM veneers.

I have attached an alternate patch (maybe a little bit rough, but to get the
idea rapidly) that sticks to the original VLDM veneers size, certainly the same
idea, but enabling a denser packing of the DP registers. By proceeding like
this (dealing with chunks instead of registers) the change in bdf/elf32-arm.c
is less intrusive.

In you patch, I am seeing also some factorization when creating the new_insn
for instance, that certainly are worth doing.

So if we could agree on a mix of the two changes, it would be great.

Best,
--C

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils

Reply via email to