asm-dude icon indicating copy to clipboard operation
asm-dude copied to clipboard

Could QuickInfo add Opcode and Description info

Open matt-elson opened this issue 7 years ago • 3 comments

QuickInfo tooltip offers the following feature, image Could QuickInfo support the following info(It is from https://github.com/HJLebbink/asm-dude/wiki/IMUL ), image

matt-elson avatar Jul 07 '18 04:07 matt-elson

I had two reasons not to add that info in the performance info.

  1. I cannot do that without handpicking the descriptions and opcodes for each combination of instruction and operand. The performance info is from Agner Fog and the descriptions are from the wiki. These sources are not very comparable. I think it would be difficult to figure out (with some Python) which description should be matched to a performance line. E.g. line with IMUL r8 and IMUL m8 should have the same description and IMUL r,r would be described differently.

  2. Other issue is that there is no obvious place to put that description on a line. The descriptions of IMUL are short compared to those of others.

If you have an idea how to solve both issue, do let me know. I do think that having the description would be useful.

HJLebbink avatar Jul 07 '18 07:07 HJLebbink

  1. Info from Agner Fog and the Wiki, respectively, don't need to be merged . Can AsmDude add option in Visual Studio's Options window? which is just like, image When "Performance" is enabled, Quickinfo displays performance info. When "Description" is enabled, Quickinfo displays the descriptions are from the wiki.
  2. There are 1457 instructions in the wiki, which descriptions are the same with Intel's manual,e.g.,
    image Can AsmDude show one line's description for each instruction? When someone need see the whole description, he can open the instruction's URL at wiki.

matt-elson avatar Jul 07 '18 08:07 matt-elson

Sorry for the unrelated question, but my AsmDude doesn't How to show the opcode when I hover over an instruction. It does show pretty much everything else. Anybody knows if there's an option that I have to enable for this? I can't figure it out.

KeyC0de avatar Sep 25 '21 15:09 KeyC0de