[PATCH] D42740: Implement a case-folding version of DJB hash

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 1 08:36:10 PST 2018


JDevlieghere added inline comments.


================
Comment at: lib/Support/UnicodeCaseFold.cpp:1
+//===---------- Support/UnicodeCaseFold.cpp -------------------------------===//
+//
----------------
And here.


================
Comment at: unittests/Support/DJBTest.cpp:1
+//===---------- llvm/unittest/Support/DJBTest.cpp -------------------------===//
+//
----------------
Same here.


================
Comment at: utils/unicode-case-fold.py:110
+
+print '//===---------- Support/UnicodeCaseFold.cpp -------------------------------===//'
+print '//'
----------------
Nit: missing `---*- C++ -*-===//` at the end. 


Repository:
  rL LLVM

https://reviews.llvm.org/D42740





More information about the llvm-commits mailing list