[PATCH] isCharInSet refactoring.

[Richard Smith]

================
Comment at: include/llvm/Support/UnicodeCharRanges.h:52-54
@@ -61,2 +51,5 @@
+  /// instances per each array of ranges, as possible.
+  UnicodeCharSet(CharRanges Ranges) : Ranges(Ranges) {
+    assert(rangesAreValid());
   }
 
----------------
It'd be great to make this `LLVM_CONSTEXPR` in the `NDEBUG` case, to avoid a guard variable for the static locals in Clang. (You'll also need to make the corresponding `ArrayRef` constructor be `LLVM_CONSTEXPR`.)


http://llvm-reviews.chandlerc.com/D1534

BRANCH
  svn

ARCANIST PROJECT
  llvm