annotate tests/mini/asm9.d @ 979:523bf4f166bc

Fix some assembler issues: The assembler was miscompiling "add" (specifically, the "add reg/mem, imm" variations). The change that caused this seems to have been made because without it, some "add"s didn't compile at all. This patch reverts the previous change, and makes sure assembler operands are remapped correctly even though the input operands auto-generated due to updating operations aren't explicitly used.
author Frits van Bommel <fvbommel wxs.nl>
date Wed, 18 Feb 2009 21:46:14 +0100
parents
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
979
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
1 module asm9;
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
2
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
3 version(X86) version = DoSome;
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
4 else version(X86_64) version = DoSome;
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
5
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
6 T add(T, T t)(T a) {
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
7 asm {
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
8 add a, t;
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
9 }
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
10 return a;
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
11 }
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
12
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
13 void main() {
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
14 version (DoSome) {
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
15 assert(add!(ubyte, 20)(10) == 30);
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
16 assert(add!(ushort, 20_000)(10_000) == 30_000);
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
17 assert(add!(uint, 2_000_000)(1_000_000) == 3_000_000);
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
18 }
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
19 version(X86_64) {
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
20 // 64-bit immediates aren't allowed on "ADD", nor are
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
21 // unsigned 32-bit ones, so make the template parameter
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
22 // fit in a 32-bit signed int.
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
23 // These values were chosen so that the lower 32-bits overflow
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
24 // and we can see the upper half of the 64-bit input increment.
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
25 auto result = add!(long, 2_000_000_000)(21_000_000_000);
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
26 assert(result == 23_000_000_000);
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
27 }
523bf4f166bc Fix some assembler issues:
Frits van Bommel <fvbommel wxs.nl>
parents:
diff changeset
28 }