Tokyo 10.2.3 Community Edition: Bug in Math-Unit?

[gammatester]

Hallo,
ich habe gestern die Tokyo Community Edition installiert :thumb: und dann (natürlich) ziemlich bald den Quellcode der Math-Unit angeschaut, und denke da ist ein Bug drin:

Delphi-Quellcode:

function GetSSEType: Cardinal;
...
{$ELSEIF Defined(CPUX86)}
asm
        PUSH    EBX
        MOV     EAX, 1
        CPUID
        XOR     EAX, EAX
        TEST    EDX, $02000000  // EDX 25 bits - SSE bit
        JZ      @@CheckSSE2
        OR      EAX, seSSE
...
@@CheckSSE42:
        TEST    ECX, $00100000  // ECX 20 bits - SSSE4.2 bit
        JZ      @@CheckPOPCNT
        OR      EAX, seSSE42
@@CheckPOPCNT:
        TEST    ECX, $00800000  // ECX 23 bits - POPCNT bit
        JZ      @@CheckAESNI
        OR      EAX, sePOPCNT
@@CheckAESNI:
        TEST    ECX, $02000000  // ECX 25 bits - AESNI bit
        JZ      @@CheckPCLMULQDQ
        OR      EAX, sePOPCNT
@@CheckPCLMULQDQ:
        TEST    ECX, $00000002  // ECX 1 bits - PCLMULQDQ bit
        JZ      @@Exit
        OR      EAX, sePCLMULQDQ
@@Exit:
        POP     EBX
end;

Und ähnlich für CPUX64. Ich glaube, da hat jemand einen Copy/Paste-Fehler gemacht und es sollte so aussehen:

Delphi-Quellcode:

@@CheckAESNI:
        TEST    ECX, $02000000  // ECX 25 bits - AESNI bit
        JZ      @@CheckPCLMULQDQ
        OR      EAX, seAESNI

Leider hat mein I3 laut CPU-Z keine Advanced Encryption Standard Instructions (AES-NI), so das ich natürlich auch von Delphi kein AES-NI (seAESNI = $0080) sehen kann. Das folgende Program

Delphi-Quellcode:

{$apptype console}
uses
  system.sysutils, system.math;
begin
  writeln(IntToHex(TestSSE,16));
end.

gibt 000000000000017F aus sowohl für 32- als auch 64-Bit und es fehlt das $80-Bit. Kann jemand mit AES-NI-Processor-Feature das als Bug bestätigen (d.h. fehlt auch bei ihm das $80-Bit)?

[Der schöne Günther]

Meine CPU hat AES-NI, aber ich verstehe nur Bahnhof. Gibt es Code den man ausführen kann und dessen Ergebnis posten soll?

[Lemmy]

Zitat:

Zitat von Der schöne Günther (Beitrag 1408068)

Meine CPU hat AES-NI, aber ich verstehe nur Bahnhof. Gibt es Code den man ausführen kann und dessen Ergebnis posten soll?

Code steht doch schon da...

ansonsten geht es mir so wie Dir, weiß aber nicht mal ob meine CPU AES-NI hat...

[Uwe Raabe]

Ich sehe das genauso. Ist ja auch ganz offensichtlich und lässt sich im Debugger leicht nachvollziehen.

Machst du einen QP-Eintrag dafür auf?

[gammatester]

Zitat:

Zitat von Uwe Raabe (Beitrag 1408070)

Machst du einen QP-Eintrag dafür auf?

Ja ich werde mich mal durchkämpfen, habe jetzt endlich die Create-Issue-Seite erreicht. Dauert wohl noch etwas bis ich damit fertig bin.

[Uwe Raabe]

Zitat:

Zitat von gammatester (Beitrag 1408072)

Ja ich werde mich mal durchkämpfen, habe jetzt endlich die Create-Issue-Seite erreicht. Dauert wohl noch etwas bis ich damit fertig bin.

Ich weiß, daß es etwas zäh sein kann, aber es ist nun mal der einzige Weg, um solche Bugs überhaupt zu beheben.

Wir müssen uns auch wohl auf eine höhere Bug-Rate in QP einstellen, jetzt wo so viele neue CE Installationen im Umlauf sind.

[gammatester]

Zitat:

Zitat von Uwe Raabe (Beitrag 1408073)

Zitat:

Zitat von gammatester (Beitrag 1408072)

Ja ich werde mich mal durchkämpfen, habe jetzt endlich die Create-Issue-Seite erreicht. Dauert wohl noch etwas bis ich damit fertig bin.

Ich weiß, daß es etwas zäh sein kann, aber es ist nun mal der einzige Weg, um solche Bugs überhaupt zu beheben.

Wir müssen uns auch wohl auf eine höhere Bug-Rate in QP einstellen, jetzt wo so viele neue CE Installationen im Umlauf sind.

Fertig: https://quality.embarcadero.com/browse/RSP-20920

PS: Community Edition erscheint noch(?) nicht, habe Professional gewählt.

[Uwe Raabe]

Zitat:

Zitat von gammatester (Beitrag 1408074)

Fertig: https://quality.embarcadero.com/browse/RSP-20920

:thumb:

Zitat:

Zitat von gammatester (Beitrag 1408074)

(allerdings mit 1, 2 Tippfehlern, kann man die korrigieren?)

Klar! Der Button "Update Content" führt in der Regel zum Ziel.

Zitat:

Zitat von gammatester (Beitrag 1408074)

PS: Community Edition erscheint noch(?) nicht, habe Professional gewählt.

Das ist in Ordnung so. Für das Bug-Management sind beide Editionen gleichwertig.

[Stevie]

Tip: mit {code:delphi} {code} kann man im JIRA Formatierung erreichen, das sieht dann gleich viel übersichtlicher aus.

[gammatester]

Zitat:

Zitat von Stevie (Beitrag 1408077)

Tip: mit {code:delphi} {code} kann man im JIRA Formatierung erreichen, das sieht dann gleich viel übersichtlicher aus.

Funktioniert bei mir irgendwie nicht. {code:delphi} malt eine gestrichelte Box, {delphi} erscheint wie es ist. Habe auch in der Formating-Hilfe keinen Hinweis darauf gefunden.

Edit: Danke, hat sich erledigt.