Правильно ли использую openMP

#include <sstream>
#include <iostream>
#include "windows.h"
#include <Windows.h>
#include <stdio.h>
#include <string>
#include <omp.h>
#include <time.h>
#include <vector>
#include <time.h>
#include <iomanip>      // std::setprecision
using namespace std;
 
typedef struct {
    int x;
    int y;
} RESULT;
 
int _tmain(int argc, _TCHAR* argv[])
{
    clock_t timer;
    timer = clock();
    stringstream strm;
    if (argc < 1)
    {
        cout << "ERROR!!!!\n";
        return 1;
    }
    SYSTEM_INFO sysinfo;
    GetSystemInfo(&sysinfo);
    int numCPU = sysinfo.dwNumberOfProcessors;
    numCPU = (numCPU == 1) ? 2 : numCPU;
    size_t number_of = 0;
    string tmp(argv[1]);
    strm << tmp;
    strm >> number_of;
    vector<RESULT> results;
    omp_set_num_threads(numCPU);     // задаём количество параллельных процессов
#pragma omp parallel for/* private(fields)   */// директива, задающая распараллеливание данного цикла
    for (int i = 0; i < number_of; ++i)
    {
//#pragma omp parallel for   /// вот здесь я не знаю нужна ли данная инструкция, если закомментирована работает программа быстрее
        for (int j = 0; j < number_of; ++j)
        {
                if ((i == j) )
                {
                    RESULT res;
                    res.x = i;
                    res.y = j;
#pragma omp critical
                    {
                        results.push_back(res);
                    }
            }
        }
    }
 
    timer = clock() - timer;
    std::cout << (double)timer / CLOCKS_PER_SEC << "\t count  == " << results.size() << std::endl;
    return 0;
}