Switch to optimized ffsl supported on more platforms

jbeich · jbeich · commit f85e81ed4180 · 2020-08-14T08:21:44.000Z
In file included from Source/GmmLib/TranslationTable/GmmAuxTable.cpp:30:
Source/GmmLib/TranslationTable/../TranslationTable/GmmUmdTranslationTable.h: In function ‘int _BitScanForward(uint32_t*, uint32_t)’:
Source/GmmLib/TranslationTable/../TranslationTable/GmmUmdTranslationTable.h:63:9: error: ‘ffsl’ was not declared in this scope
     i = ffsl(mask);
         ^~~~
Source/GmmLib/TranslationTable/../TranslationTable/GmmUmdTranslationTable.h:63:9: note: suggested alternative: ‘ffs’
     i = ffsl(mask);
         ^~~~
         ffs
diff --git a/Source/GmmLib/TranslationTable/GmmUmdTranslationTable.h b/Source/GmmLib/TranslationTable/GmmUmdTranslationTable.h
@@ -60,12 +60,7 @@ static inline int _BitScanForward(uint32_t *index, uint32_t mask)
 {
     int i;
 
-#ifdef __ANDROID__
-    i = ffs(mask);
-#else
-    i = ffsl(mask);
-#endif
-
+    i = __builtin_ffsl(mask);
     if(i > 0)
     {
         *index = (uint32_t)(i - 1);

Original file line number	Diff line number	Diff line change
`@@ -60,12 +60,7 @@ static inline int _BitScanForward(uint32_t *index, uint32_t mask)`
`60`	`60`	`{`
`61`	`61`	`int i;`
`62`	`62`
`63`		`-#ifdef __ANDROID__`
`64`		`- i = ffs(mask);`
`65`		`-#else`
`66`		`- i = ffsl(mask);`
`67`		`-#endif`
`68`		`-`
	`63`	`+ i = __builtin_ffsl(mask);`
`69`	`64`	`if(i > 0)`
`70`	`65`	`{`
`71`	`66`	`*index = (uint32_t)(i - 1);`