[LLVMdev] DataTypes.h for Visual C

Jochen Wilhelmy j.wilhelmy at arcor.de
Fri Jun 25 00:46:42 PDT 2010

>>> What's that extra code path?
>> #ifndef _MSC_VER
>> ...
>> #else /* _MSC_VER */
>> /* Visual C++ doesn't provide standard integer headers, but it does provide
>>     built-in data types. */
>> ... extra code path ...
>> #endif
> Sorry, that doesn't show an extra code path. DataTypes.h contains stuff
> specific for MSVC and stuff for the rest of compilers. The
> #ifndef...#else does just that. Maybe you are reading #ifndef as #ifdef?
I just wanted to show you what I mean by the extra code path (the 
compiler takes
one part of code or another dependent on a define), but obviously you 
don't call it code path.
How do you call it?

> Adding just another config option (and one that is speficic of one
> compiler) seems overkill to me and would constitute a precedent for
> similar cases on the future. I try to keep the config option list as
> short as possible.
> Protecting the defines, OTOH, is okay.
Yes, protecting the defines will be enough.

- Jochen

More information about the llvm-dev mailing list