src/Makefile.am | 4 ++-- src/hb-set-private.hh | 7 +++---- 2 files changed, 5 insertions(+), 6 deletions(-)
New commits: commit 2903b2f357eb495c9badac1b2f74e65e1d1075bd Author: Behdad Esfahbod <[email protected]> Date: Wed Feb 14 15:46:27 2018 -0800 Another linker tweaking Trying to fix this warning: cc1: warning: command line option ‘-fno-threadsafe-statics’ is valid for C++/ObjC++ but not for C Let's see what the clang bots think of it. diff --git a/src/Makefile.am b/src/Makefile.am index 8288bd7f..e3915bc8 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -37,8 +37,8 @@ HBNOLIBCXXCFLAGS = else # Make sure we don't link to libstdc++ # No threadsafe statics in C++ as we do it ourselves -HBCFLAGS += -fno-threadsafe-statics -fno-exceptions -HBNOLIBCXXFLAGS = -fno-rtti +HBCFLAGS += -fno-exceptions +HBNOLIBCXXFLAGS = -fno-threadsafe-statics -fno-rtti endif if HAVE_OT commit e1f1f96577a80846749782b6e199a661f4189ec0 Author: Behdad Esfahbod <[email protected]> Date: Wed Feb 14 15:44:48 2018 -0800 [set] Tune size again Switch from a 128+64 hierarchy, to 32+32. New page size is 1024. This should make set iteration faster, without affecting other op performances much. diff --git a/src/hb-set-private.hh b/src/hb-set-private.hh index 7282b079..0c73d32d 100644 --- a/src/hb-set-private.hh +++ b/src/hb-set-private.hh @@ -177,11 +177,11 @@ struct hb_set_t return 0; } - static const unsigned int PAGE_BITS = 8192; /* Use to tune. */ + typedef uint32_t elt_t; + static const unsigned int ELT_BITS = sizeof (elt_t) * 8; + static const unsigned int PAGE_BITS = ELT_BITS * ELT_BITS; /* 1024. Use to tune. */ static_assert ((PAGE_BITS & ((PAGE_BITS) - 1)) == 0, ""); - typedef uint64_t elt_t; - #if 0 && HAVE_VECTOR_SIZE /* The vectorized version does not work with clang as non-const * elt() errs "non-const reference cannot bind to vector element". */ @@ -192,7 +192,6 @@ struct hb_set_t vector_t v; - static const unsigned int ELT_BITS = sizeof (elt_t) * 8; static const unsigned int ELT_MASK = ELT_BITS - 1; static const unsigned int BITS = sizeof (vector_t) * 8; static const unsigned int MASK = BITS - 1; _______________________________________________ HarfBuzz mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/harfbuzz
