Code review comment for lp:~epics-core/epics-base/3.15-buildCompilerSpecific

Andrew Johnson (anj) wrote :

This doesn't fully cover all our architectures yet.

Both darwin and linux can build Base using clang, but when I use clang on Linux with the gcc header it doesn't hit the #error in compiler/gcc/compilerSpecific.h since clang defines __GNUC__ anyway. There needs to be an easy way to make that switch (including setting CC and CCC) based on a single CONFIG_SITE setting.

Jeff: I'm happy to work on implementing these fixes back at ANL. Have you based your atomic branch on this one? I'd like to drop your 12229 merge commit, but that would probably mess you up if I did.

- Andrew

« Back to merge proposal