From 816e9681d3eb86818be20fde0f1f63d25b8ebbe3 Mon Sep 17 00:00:00 2001 From: George Burgess IV Date: Thu, 25 Aug 2016 01:29:55 +0000 Subject: [PATCH] Update a comment. r279696, which changed `LLVM_CONSTEXPR AliasAttr` to `const AliasAttr`, made this comment make less sense. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@279699 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Analysis/AliasAnalysisSummary.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/lib/Analysis/AliasAnalysisSummary.cpp b/lib/Analysis/AliasAnalysisSummary.cpp index 09a6b8217a08..2b4879453beb 100644 --- a/lib/Analysis/AliasAnalysisSummary.cpp +++ b/lib/Analysis/AliasAnalysisSummary.cpp @@ -15,9 +15,8 @@ const unsigned AttrFirstArgIndex = 4; const unsigned AttrLastArgIndex = NumAliasAttrs; const unsigned AttrMaxNumArgs = AttrLastArgIndex - AttrFirstArgIndex; -// NOTE: These aren't AliasAttrs because bitsets don't have a constexpr -// ctor for some versions of MSVC that we support. We could maybe refactor, -// but... +// It would be *slightly* prettier if we changed these to AliasAttrs, but it +// seems that both GCC and MSVC emit dynamic initializers for const bitsets. using AliasAttr = unsigned; const AliasAttr AttrNone = 0; const AliasAttr AttrEscaped = 1 << AttrEscapedIndex;