janestreet / core_extended

Jane Street Capital's standard library overlay
MIT License
34 stars 9 forks source link

rdtscp clobbers %rcx #9

Closed softminus closed 8 years ago

softminus commented 8 years ago

the asm stanza in the rdtscp stub should specify that %rcx is clobbered by the rdtscp instruction (see page 113 of Intel 64 and IA-32 Architectures Software Developer’s Manual)

ghost commented 8 years ago

Thanks, I submitted a feature for review internally

ghost commented 8 years ago

Fixed by d333da7b49918e2bbb4c62bc20bd1566e25dc69f