steam-frame/FEX-Emu--FEX
1
0 Fork 0
mirror of https://github.com/FEX-Emu/FEX.git synced 2026-10-14 18:00:46 +02:00
Code Issues Packages Projects Releases 67 Wiki Activity

Replaces FEX's usage of classic C array element count with std::size

Browse Source 
Just learned that this is a c++17 feature and it's a nice little
cleanup.

Only one remains in our codebase which can't easily be replaced
This commit is contained in:
Ryan Houdek committed 2021-03-28 05:39:35 -07:00
1 parent 432b6fccf2
commit 650877adca
15 files changed
+32 -32
Download Patch File Download Diff File Expand all files Collapse all files

No files matched your search

Source/Tests/UnitTestGenerator.cpp
+6 -6
View File
@@ -193,7 +193,7 @@ void GeneratePrimaryTable() {
bool AddressSizePrefix = false;
auto DoNormalOps = [&](const char *NameSuffix, auto Inserter, std::optional<std::function<void()>> ModRMInserter, uint8_t REX = 0) {
for (size_t OpIndex = 0; OpIndex < (sizeof(BaseOps) / sizeof(BaseOps[0])); ++OpIndex) {
for (size_t OpIndex = 0; OpIndex < std::size(BaseOps); ++OpIndex) {
auto &Op = BaseOps[OpIndex];
if (Op.Type == TYPE_INST) {
if (Op.Flags & InstFlags::FLAGS_SETS_RIP ||
@@ -764,7 +764,7 @@ void GeneratePrimaryGroupTable() {
CurrentPrefix = "PrimaryGroup";
auto DoNormalOps = [&](const char *NameSuffix, auto SkipCheck, auto Inserter, auto &Table, std::optional<std::function<void(uint8_t)>> ModRMInserter, uint8_t REX = 0) {
for (size_t OpIndex = 0; OpIndex < (sizeof(Table) / sizeof(Table[0])); ++OpIndex) {
for (size_t OpIndex = 0; OpIndex < std::size(Table); ++OpIndex) {
auto &Op = Table[OpIndex];
if (Op.Type == TYPE_INST) {
if (Op.Flags & InstFlags::FLAGS_SETS_RIP ||
@@ -1147,7 +1147,7 @@ void GenerateSecondaryTable() {
CurrentPrefix = "Secondary";
auto DoNormalOps = [&](const char *NameSuffix, auto SkipCheck, auto Inserter, std::optional<std::function<void()>> ModRMInserter, uint8_t REX = 0) {
for (size_t OpIndex = 0; OpIndex < (sizeof(SecondBaseOps) / sizeof(SecondBaseOps[0])); ++OpIndex) {
for (size_t OpIndex = 0; OpIndex < std::size(SecondBaseOps); ++OpIndex) {
auto &Op = SecondBaseOps[OpIndex];
if (Op.Type == TYPE_INST) {
if (Op.Flags & InstFlags::FLAGS_SETS_RIP ||
@@ -1542,7 +1542,7 @@ void GenerateSecondaryGroupTable() {
CurrentPrefix = "SecondaryGroup";
auto DoNormalOps = [&](const char *NameSuffix, auto SkipCheck, auto Inserter, std::optional<std::function<void(uint8_t)>> ModRMInserter, uint8_t REX = 0) {
for (size_t OpIndex = 0; OpIndex < (sizeof(SecondInstGroupOps) / sizeof(SecondInstGroupOps[0])); ++OpIndex) {
for (size_t OpIndex = 0; OpIndex < std::size(SecondInstGroupOps); ++OpIndex) {
auto &Op = SecondInstGroupOps[OpIndex];
if (Op.Type == TYPE_INST) {
if (Op.Flags & InstFlags::FLAGS_SETS_RIP ||
@@ -1937,7 +1937,7 @@ void GenerateSSEInstructions() {
bool AddressSizePrefix = false;
auto DoNormalOps = [&](const char *NameSuffix, auto SkipCheck, auto Inserter, auto &Table, std::optional<std::function<void()>> ModRMInserter, uint8_t REX = 0) {
for (size_t OpIndex = 0; OpIndex < (sizeof(Table) / sizeof(Table[0])); ++OpIndex) {
for (size_t OpIndex = 0; OpIndex < std::size(Table); ++OpIndex) {
auto &Op = Table[OpIndex];
if (Op.Type == TYPE_INST) {
if (Op.Flags & InstFlags::FLAGS_SETS_RIP ||
@@ -2419,7 +2419,7 @@ int main(int argc, char **argv, char **const envp) {
FILE *fp = fopen(Filename.c_str(), "wbe");
fprintf(fp, "HEX, Name, Num Times compiled\n");
for (size_t OpIndex = 0; OpIndex < (sizeof(Table) / sizeof(Table[0])); ++OpIndex) {
for (size_t OpIndex = 0; OpIndex < std::size(Table); ++OpIndex) {
auto &Op = Table[OpIndex];
if (Op.Type == TYPE_INST) {
fprintf(fp, "0x%zx, %s, %d\n", OpIndex, Op.Name, Op.NumUnitTestsGenerated);