2

I have a loop like below, I think the problem is with the inner for loops marked "See here". Whats the problem with that? 

// measure time to write different sizes of data
for (int i = 0; i < sizeof(sizes)/sizeof(int); i++) {
    lengthMod = sizes[i]/sizeof(int) - 1;

    start = wall_clock_time();

    for (unsigned int j = 0; j < 10; j++) // << See here!!!
        tmp = 1;

    // force any write back cache to flush. read from other data source
    for (unsigned int j = 0; j < REPS; j++) // << Or here!!!
        tmp = j;

    end = wall_clock_time();
    timeTaken = ((float)(end - start))/1000000000;
    fprintf(stderr, "%d, %1.2f \n", sizes[i]/1024, ((float)(end - start))/1000000000);
}

You can see the full source on GitHub ~line 32

g++ -O3 cache-write.cpp -o cache-write -lrt
cache-write.cpp: In function ‘int main()’:
cache-write.cpp:36:27: error: expected primary-expression before ‘;’ token
cache-write.cpp:36:27: error: expected ‘)’ before ‘;’ token
cache-write.cpp:36:29: error: name lookup of ‘j’ changed for ISO ‘for’ scoping [-fpermissive]
cache-write.cpp:36:29: note: (if you use ‘-fpermissive’ G++ will accept your code)
cache-write.cpp:36:32: error: expected ‘;’ before ‘)’ token
cache-write.cpp:44:11: error: ‘data1’ was not declared in this scope
make: *** [cache-write] Error 1
4

2 回答 2

4

不要在声明 REPS 的地方使用 #define，而是使用

const unsigned int REPS = whatever;

通常在 C++ 中避免使用#define，因为它很容易导致您遇到的那种问题。

于 2012-10-06T12:05:31.080 回答
3

REPS 定义的末尾有一个分号：

#define REPS 128 * MB;
于 2012-10-06T12:03:49.170 回答