build failed about atomic

[tensor-tang]

hi
just encounter an issue with latest commit 6f9a9a9

paddle/framework/operator.cc: In member function ‘void paddle::framework::OperatorBase::GenerateTemporaryNames()’:
paddle/framework/operator.cc:194:10: error: ‘atomic’ in namespace ‘std’ does not name a type
static std::atomic<size_t> gUniqId(0UL);

CentOS Linux release 7.3.1611 (Core)

gcc version

g++ (GCC) 4.8.5 20150623 (Red Hat 4.8.5-11)
Copyright (C) 2015 Free Software Foundation, Inc.

It works after added #include <atomic>

[luotao1]

Both @xieYK and me couldn't reappear this problem.
What's your cmake option?

[tensor-tang]

My cmake options:
cmake .. -DWITH_GPU=OFF -DWITH_STYLE_CHECK=OFF -DWITH_TESTING=ON -DWITH_TIMER=OFF -DWITH_SWIG_PY=ON -DWITH_DSO=OFF -DWITH_MKLDNN=ON -DWITH_MKLML=ON

[luotao1]

Maybe the reason of CentOS 7.3, but it's no harm to add #include <atomic>. Could you create a PR to fix it ?

[tensor-tang]

OK, sure, will update soon.

[luotao1]

I reappear this compiler problem in ONLYCPU mode. Should our teamcity check ONLYCPU mode? @helinwang

[helinwang]

@luotao1 Currently we have CPU nightly build, which build with WITH_GPU=OFF. I am curious why we have another flag ONLYCPU, feels like it duplicates with WITH_GPU=OFF.

[luotao1]

@helinwang Oh, the ONLYCPU mode means WITH_GPU=OFF. And how do think #4476 as @emailweixu suggests?

[helinwang]

That's a good idea (replied details in that issue).