Replaced shift and add sequence with single multiply in hash function

Archaic optimisation to think a sequence of instructions with
pipeline-stalling dependencies beats a single imul.
Compilers can still make the substitution with a shift and add if they
want to for a particular platform.

Change-Id: I313a5f98eae665f0fabd266fee09c078547be2ec
Signed-off-by: Andrew Cox <andrew.cox@partner.samsung.com>

diff --git a/dali/internal/common/dali-hash.cpp b/dali/internal/common/dali-hash.cpp
index f154b14393b52096a228917e414715bad50f2b80..410141470ac850eb223c7cad58ebd217cd52487d 100644 (file)
--- a/dali/internal/common/dali-hash.cpp
+++ b/dali/internal/common/dali-hash.cpp
@@ -33,7 +33,7 @@ inline void HashShader( const char* string, unsigned long& hash )
 {
   while( int c = *string++ )
   {
-    hash = ((hash << 5) + hash) + c; /* hash * 33 + c */
+    hash = hash * 33 + c;
   }
 }
 } // un-named namespace