Re: [PATCH] D22439: Add missing includes.

2016-09-14 Thread Vassil Vassilev via cfe-commits
v.g.vassilev added a comment. The 'atomic' bug was fixed I assume as part of https://llvm.org/bugs/show_bug.cgi?id=28794 Other includes landed in r281450. https://reviews.llvm.org/D22439 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http

Re: [PATCH] D22439: Add missing includes.

2016-07-20 Thread Richard Smith via cfe-commits
rsmith added a comment. ManagedStatic.h already includes , so the additional includes of don't seem appropriate to me. Looks like this may have hit a bug in enum merging? https://reviews.llvm.org/D22439 ___ cfe-commits mailing list cfe-commits@li