[PATCH] D27312: Remove MSVC workaround for AlignedCharArray (NFC)

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 1 12:02:08 PST 2016


mehdi_amini updated this revision to Diff 79964.
mehdi_amini added a comment.

Thanks bkramer! Let's just improve the comment then?


https://reviews.llvm.org/D27312

Files:
  llvm/include/llvm/Support/AlignOf.h


Index: llvm/include/llvm/Support/AlignOf.h
===================================================================
--- llvm/include/llvm/Support/AlignOf.h
+++ llvm/include/llvm/Support/AlignOf.h
@@ -24,7 +24,7 @@
 ///
 /// This template is used to explicitly build up a collection of aligned
 /// character array types. We have to build these up using a macro and explicit
-/// specialization to cope with old versions of MSVC where only an
+/// specialization to cope with MSVC (at least till 2015) where only an
 /// integer literal can be used to specify an alignment constraint. Once built
 /// up here, we can then begin to indirect between these using normal C++
 /// template parameters.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27312.79964.patch
Type: text/x-patch
Size: 691 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161201/1c126b41/attachment.bin>


More information about the llvm-commits mailing list