C++ указательная оптимизация

[loganhackerdff]

Раньше у меня был слабый ноутбук, и это оставило на мне душевную травму.
Теперь я все оптимизирую.
В функции я всё что весит больше 4 байт, я передаю по указателю (даже не по ссылке)
Я не использую range based циклы, потому-что они медленнее чем перебор с переменной.
Я правильно делаю?
Или так похож на старпёра

 

[manukhov]

Можно динамической памятью пользоваться, с new и delete

 

[kin4stat]

Можно динамической памятью пользоваться, с new и delete

А можно не говнокодить, и использовать умные указатели

 

[traceattack]

Дабл через указатель.

Люди вводят новые конструкции в стандарт. Тем временем...

 

[SR_team]

я передаю по указателю (даже не по ссылке)

в скомпилированном коде это одно и то же

Я не использую range based циклы, потому-что они медленнее чем перебор с переменной.

не медленнее. Оба варианта компилятор оптимизирует из вида:

// перебор с переменной
for ( int i = 0; i < count; ++i ) {
    auto &el = arr[i];
    // ...
}
// range-based
for ( auto && el : arr ) {
    // ...
}

в:

for ( auto el = *arr; el < count * sizeof(el); el += sizeof(el) ) {
    // ...
}

 

[loganhackerdff]

в скомпилированном коде это одно и то же

не медленнее. Оба варианта компилятор оптимизирует из вида:

// перебор с переменной
for ( int i = 0; i < count; ++i ) {
    auto &el = arr[i];
    // ...
}
// range-based
for ( auto && el : arr ) {
    // ...
}

в:

for ( auto el = *arr; el < count * sizeof(el); el += sizeof(el) ) {
    // ...
}

Я так понял конструктор копирования по ссылке копирует всё из объекта , а указатель передаёт весь объект по ссылке?
Похоже нет, посмотри сам, второй вариант быстрее

#define _CRT_SECURE_NO_WARNINGS
#include <iostream>
#include "vector"
//#include "string"
#include "chrono"
//#include "C:\LoganSnipps.h"
using namespace std;

struct A
{
    int a, b, c, d, h, n, q = {};
};

int main()
{
    while (true)
    {
        {
          
            std::vector<A> a(100);
            auto st = chrono::high_resolution_clock::now();
            for (auto&& val : a)
            {
                //sizeof(val) =  28
                std::cout << val.a << ' ';
            }
            chrono::duration<float> dura = chrono::high_resolution_clock::now() - st;
            cout << endl;
            cout << dura.count() << endl;
            cout << "It was &&" << endl;
        }
        cout << endl;
        {  
            std::vector<A> a(100);
            auto st = chrono::high_resolution_clock::now();
            for (size_t i = 0; i < a.size(); i++)
            {
                //sizeof(i) =  4
                //sizeof(a[i]) = 28
                std::cout << a[i].a << ' ';
            }
            chrono::duration<float> dura = chrono::high_resolution_clock::now() - st;
            cout << endl;
            cout << dura.count() << endl;
            cout << "It was size_t i = 0" << endl;
        }
        getchar();
    }

}

 

[SR_team]

Я так понял конструктор копирования по ссылке копирует всё из объекта , а указатель передаёт весь объект по ссылке?
Похоже нет, посмотри сам, второй вариант быстрее

#define _CRT_SECURE_NO_WARNINGS
#include <iostream>
#include "vector"
//#include "string"
#include "chrono"
//#include "C:\LoganSnipps.h"
using namespace std;

struct A
{
    int a, b, c, d, h, n, q = {};
};

int main()
{
    while (true)
    {
        {
         
            std::vector<A> a(100);
            auto st = chrono::high_resolution_clock::now();
            for (auto&& val : a)
            {
                //sizeof(val) =  28
                std::cout << val.a << ' ';
            }
            chrono::duration<float> dura = chrono::high_resolution_clock::now() - st;
            cout << endl;
            cout << dura.count() << endl;
            cout << "It was &&" << endl;
        }
        cout << endl;
        { 
            std::vector<A> a(100);
            auto st = chrono::high_resolution_clock::now();
            for (size_t i = 0; i < a.size(); i++)
            {
                //sizeof(i) =  4
                //sizeof(a[i]) = 28
                std::cout << a[i].a << ' ';
            }
            chrono::duration<float> dura = chrono::high_resolution_clock::now() - st;
            cout << endl;
            cout << dura.count() << endl;
            cout << "It was size_t i = 0" << endl;
        }
        getchar();
    }

}

Тебе поток вывода (std::cout) глаз замылил. Если поменять циклы местами, то результаты тоже поменяются в противоположную сторону

 

[SR_team]

Вот что нагенерировал компилятор для range based:

А вот что он нагенирировал для цикла с перебором:

Как видишь код в обоих случаях одинаковый. Отличие есть только в том, что в цикле с переменной вместо размера вектора стоит переменная, в которую размер был прочитан из структуры вектора, но в кейсах с заранее не известной длиной у range based будет тоже чтение из структуры. Так что в скорости они не отличаются.

 

[kin4stat]

Вот что нагенерировал компилятор для range based:
Посмотреть вложение 71494
А вот что он нагенирировал для цикла с перебором:
Посмотреть вложение 71495

Как видишь код в обоих случаях одинаковый. Отличие есть только в том, что в цикле с переменной вместо размера вектора стоит переменная, в которую размер был прочитан из структуры вектора, но в кейсах с заранее не известной длиной у range based будет тоже чтение из структуры. Так что в скорости они не отличаются.

На godbolt.org можно смотреть в реалтайме asm код, который генерируют разные компиляторы с разными флагами компиляции

 

[SR_team]

На godbolt.org можно смотреть в реалтайме asm код, который генерируют разные компиляторы с разными флагами компиляции

Знаю, но asm менее нагляден, по этому я вставил скрин с псевдокодом из IDA

 

[kin4stat]

Знаю, но asm менее нагляден, по этому я вставил скрин с псевдокодом из IDA

А, ты так сделал, понял.