Enhance drm command, add drm xmm0 functionality

[koyaan]

This is for the proposal in #15213
Tests: https://github.com/radareorg/radare2-regressions/pull/1977

[radare]

i was thinking in supporting this in the r2 way.. so using a subcommand of drm instead of a special syntax for the arg. like: drmw drmh, drmq.. what do you think?

thanks for the contrib!

[koyaan]

That is a good idea, i will try to implement that! Is there a way to alias for example dmmb xmm0 to dmm xmm0 * 8 or should I remove the wildcard functionality?

[radare]

you can call the same command again with the wildcard syntax from the subcommand one if you like this way it will support both syntaxes.. is this a gdb thing ?

…

On 7 Oct 2019, at 12:30, Armin Weihbold ***@***.***> wrote: That is a good idea, i will try to implement that! Is there a way to alias for example dmmb xmm0 to dmm xmm0 * 8 or should I remove the wildcard functionality? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#15214?email_source=notifications&email_token=AAG75FQ2RHTQ63HNCA5H323QNMFU3A5CNFSM4I56PJ7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAPZLEA#issuecomment-538940816>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAG75FU3Y44R2MOGRFYZQMDQNMFU3ANCNFSM4I56PJ7A>.

[koyaan]

No I was just looking for way to have index unspecified while specifying a size without breaking existing functionality.

[radare]

then i would go for just supporting the r2 subcommand way which is more consistent with the rest of r2

[koyaan]

I added the shorthand versions now, also updated tests.

[radare]

.

[radare]

Can u update the other x64 profiles. Too? Not just the linux one

[koyaan]

Which ones? Windows does its own thing for example

[koyaan]

Removed accidental files, and adjusted C-style, also added [?] indicator to help

[koyaan]

now mmx registers also work as expected

[0x004000b0]> drm mm0
0x00 0x70 0xa0 0x8a 0x36 0xfd 0xec 0x91
0x7000 0x8aa0 0xfd36 0x91ec
0x8aa07000 0x91ecfd36
0x91ecfd368aa07000

[radare]

The changes proposed looks good to me. Thanks!!