Skip to content

Commit

Permalink
fix(libdmr): strip LoongArch out of Platform::Mips
Browse files Browse the repository at this point in the history
LoongArch is not MIPS-based Loongson and exhibits significantly different
performance characteristics (try ~50 - 100% faster with current hardware
versus 3A4000).

There is no need to strip out functionalities in order to obtain acceptable
performance. The current 3A6000 processors can easily keep up with 10th Gen
Intel i3 processors.
  • Loading branch information
MingcongBai committed Aug 2, 2024
1 parent 8b6d252 commit cf8bfc3
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions src/libdmr/compositing_manager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -64,8 +64,7 @@ class PlatformChecker
qInfo() << "match shenwei";
_pf = Platform::Alpha;

} else if (machine.find("mips") != string::npos
|| machine.find("loongarch64") != string::npos) { // loongson
} else if (machine.find("mips") != string::npos) { // MIPS-based Loongson
qInfo() << "match loongson";
_pf = Platform::Mips;
} else if (machine.find("aarch64") != string::npos) { // ARM64
Expand Down
2 changes: 1 addition & 1 deletion src/libdmr/compositing_manager.h
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ namespace dmr {
enum Platform {
Unknown,
X86, // intel & amd
Mips, // loongson
Mips, // MIPS-based Loongson
Alpha, // sunway
Arm64
};
Expand Down

0 comments on commit cf8bfc3

Please sign in to comment.