Re: [PATCH] D14259: The maximum alignment of std::aligned_storage applies to all Windows compilers

2016-09-26 Thread Marshall Clow via cfe-commits 
mclow.lists accepted this revision.
mclow.lists added a comment.

I guess this is OK. I'd rather not have naked `_WIN32` references outside of 
<__config>, but this is not the first one.


https://reviews.llvm.org/D14259



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D14259: The maximum alignment of std::aligned_storage applies to all Windows compilers

2016-09-26 Thread Eugene Zelenko via cfe-commits 
Eugene.Zelenko added a subscriber: Eugene.Zelenko.
Eugene.Zelenko added a comment.

Looks like patch was not committed.


https://reviews.llvm.org/D14259



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D14259: The maximum alignment of std::aligned_storage applies to all Windows compilers

2015-11-02 Thread Saleem Abdulrasool via cfe-commits 
compnerd added a subscriber: compnerd.
compnerd accepted this revision.
compnerd added a reviewer: compnerd.
compnerd added a comment.
This revision is now accepted and ready to land.

Yeah, this is a COFF limitation.  Its unfortunate we don't have a better way to 
detect COFF targets.


http://reviews.llvm.org/D14259



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits