[PATCH] D15514: Bump up MAX_SUBTARGET_FEATURES

Weiming Zhao via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 14 18:35:56 PST 2015


weimingz removed rL LLVM as the repository for this revision.
weimingz updated this revision to Diff 42811.
weimingz added a comment.

Agree. 128 makes more sense.


http://reviews.llvm.org/D15514

Files:
  include/llvm/MC/SubtargetFeature.h

Index: include/llvm/MC/SubtargetFeature.h
===================================================================
--- include/llvm/MC/SubtargetFeature.h
+++ include/llvm/MC/SubtargetFeature.h
@@ -30,7 +30,7 @@
 // A container class for subtarget features.
 // This is convenient because std::bitset does not have a constructor
 // with an initializer list of set bits.
-const unsigned MAX_SUBTARGET_FEATURES = 96;
+const unsigned MAX_SUBTARGET_FEATURES = 128;
 class FeatureBitset : public std::bitset<MAX_SUBTARGET_FEATURES> {
 public:
   // Cannot inherit constructors because it's not supported by VC++..


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D15514.42811.patch
Type: text/x-patch
Size: 604 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151215/18717dd8/attachment.bin>


More information about the llvm-commits mailing list