[PATCH] D22439: Add missing includes.

2016-07-16 Thread Vassil Vassilev via cfe-commits
v.g.vassilev created this revision. v.g.vassilev added a reviewer: rsmith. v.g.vassilev added a subscriber: cfe-commits. I am not sure if we should #include but the instantiation of `operator*` of `ManagedStatic` needs `std::memory_order_aquire`. Modules builds error out with: `In module 'LLVM

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