Testp and github

News and research about CPU microarchitecture and software optimization
Post Reply
alto_gmx
Posts: 1
Joined: 2024-10-31, 11:43:15

Testp and github

Post by alto_gmx » 2024-10-31, 11:50:04

Why don't you put it on github so people could contribute?
Latest change clearly has a bug. You added `case 0x8E:` twice inside GetProcessorFamily. This is a compilation error. Do you make changes blindly without even compiling?
```
1>------ Build started: Project: PMCTest, Configuration: Debug x64 ------
1>CCounters.cpp
1>D:\work-pps\testp\pmc\CCounters.cpp(883,18): error C2196: case value '142' already used
1>Done building project "PMCTest.vcxproj" -- FAILED.
```

(as a side note, this ancient forum software is just too old: you cannot copy-paste screenshot for a diff for the change I'm referring to)

agner
Site Admin
Posts: 76
Joined: 2019-12-27, 18:56:25
Contact:

Re: Testp and github

Post by agner » 2024-10-31, 15:32:35

Good idea. I will put testp on github when I get the time. I don't have the time and resources to test every new microprocessor family on the market, so it will be good if other people can help.

Sorry for the bug. I have removed the extra case label.

Post Reply