Fix -Wbitwise-instead-of-logical in 5 files starting w/ fbpcs/emp_gam…

…es/pcf2_attribution/AttributionRule_impl.h (#9310)

Summary:
Pull Request resolved: #9310

X-link: facebookincubator/dynolog#89

X-link: facebookresearch/fbpcf#462

X-link: facebookresearch/fbpcs#2016

With LLVM-15, `&&` and `||` are required for boolean operands, rather than `&` and `|` which can be confused for bitwise operations. Fixing such ambiguity helps makes our code more readable.

 - If you approve of this diff, please use the "Accept & Ship" button :-)

Differential Revision: D42347735

fbshipit-source-id: 3ec9c8af880cfce6490787662a4c3bc28f990bc7

hphp/runtime/base/runtime-option.cpp

@@ -1416,13 +1416,17 @@ static std::vector<std::string> getTierOverwrites(IniSetting::Map& ini,
     // Check the patterns one by one so they all get evaluated; otherwise, when
     // using "&&" in a single expression with multiple patterns, if an earlier
     // one fails to match, the later one would be reported as unused.
+    #pragma clang diagnostic push
+    #pragma clang diagnostic ignored "-Wbitwise-instead-of-logical"
     auto matched = true;
     matched &= Config::matchHdfPattern(hostname, ini, hdf, "machine");
     matched &= Config::matchHdfPattern(tier, ini, hdf, "tier");
     matched &= Config::matchHdfPattern(task, ini, hdf, "task");
     matched &= Config::matchHdfPattern(tiers, ini, hdf, "tiers", "m");
     matched &= Config::matchHdfPattern(tags, ini, hdf, "tags", "m");
     matched &= Config::matchHdfPattern(cpu, ini, hdf, "cpu");
+    #pragma clang diagnostic pop
+
     return matched;
   };
 
@@ -1433,10 +1437,14 @@ static std::vector<std::string> getTierOverwrites(IniSetting::Map& ini,
     // Check the patterns one by one so they all get evaluated; otherwise, when
     // using "&&" in a single expression with multiple patterns, if an earlier
     // one fails to match, the later one would be reported as unused.
+    #pragma clang diagnostic push
+    #pragma clang diagnostic ignored "-Wbitwise-instead-of-logical"
     auto matched = true;
     matched &= checkPatterns(hdf);
     matched &= !hdf.exists("exclude") || !checkPatterns(hdf["exclude"]);
     matched &= matchShard(enableShards, hostname, ini, hdf, messages);
+    #pragma clang diagnostic pop
+
     return matched;
   };
 

hphp/runtime/ext/hsl/hsl_locale_libc_ops.cpp

@@ -331,7 +331,7 @@ String HSLLocaleLibcOps::replace_every_nonrecursive_ci(const String& haystack,
     haystack,
     replacements,
     /* to_t = */ id,
-    /* from_t = */ id, 
+    /* from_t = */ id,
     /* normalize = */ [](String* s) {},
     [](String* s) {
       *s = HHVM_FN(strtolower)(*s);