kimgr added a comment.
I've seen this in the wild:
#define LOG(m) \
{ \
std::ostringstream os; \
os << m << "\n"; \
LogWrite(os.str()); \
}
auto os = GetOSName();
LOG("The OS is " << os);
It looks like your patch would miss this case. Not sure if current Clang
catches it either, though, I don't remember the exact symptoms when we found
this.
https://reviews.llvm.org/D35783
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
