use logging function name comparator for Debug.TraceFunctions

Summary:
use fstrcmp_slice(), which logs case collisions, instead of bstrcaseeq()
since we are comparing function names

Reviewed By: ricklavoie

Differential Revision: D52943301

fbshipit-source-id: 1ef5b54561ba8a8f413a1b3ce1cf5fdef2fb0495

Author: edwinsmith

hphp/runtime/vm/jit/mcgen-translate.cpp

@@ -104,7 +104,7 @@ CompactVector<Trace::BumpRelease> bumpTraceFunctions(const Func* func) {
     if (it == RuntimeOption::TraceFunctions.end()) return opt();
     folly::StringPiece name = *it;
     if (name.size() >= funcName.size() &&
-        bstrcaseeq(name.data(), funcName.data(), funcName.size())) {
+        fstrcmp_slice(funcName, name.subpiece(0, funcName.size())) == 0) {
       if (name.size() == funcName.size()) return def();
       if (name[funcName.size()] != ';') return opt();
       name.advance(funcName.size() + 1);