rnk added a comment. Fix seems reasonable.
================ Comment at: lib/Lex/PPMacroExpansion.cpp:115 + "only built-ins should have an entry here"); + assert(!OldMD->getPrevious() && "builtin should only have a singe entry"); + ED->setPrevious(OldMD); ---------------- "single" entry ================ Comment at: test/PCH/builtin-macro.c:29 + +const char s[] = __DATE__ " " __TIME__ " " __TIMESTAMP__; + ---------------- This test doesn't seem to fail if `__DATE__` expands to something. I removed `-D__DATE__=` from the command line and it passes. Can we find a way to make it fail? `int x = __DATE__ 42;`? https://reviews.llvm.org/D27545 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits