What's the goal here? It seems strange to allow GNU builtins in MS mode but not MS builtins in GNU mode.
================ Comment at: include/clang/Basic/Builtins.def:674 @@ +673,3 @@ +// Microsoft builtins. These are only active with -fms-extensions. +// They are not library builtins despire our use of LIBBUILTIN. +LIBBUILTIN(__assume, "vb", "n", 0, ALL_MS_LANGUAGES) ---------------- This isn't particularly great. Can you find a cleaner approach? Maybe: #ifndef RESTRICTED_BUILTIN #define RESTRICTED_BUILTIN(a, b, c, d) BUILTIN(a, b, c) #endif ... and then explicitly handle this in the one place where we care (in Builtins.cpp)? (Better macro name welcome!) http://llvm-reviews.chandlerc.com/D2128 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
