[gf] Remove C4018 warnings from GfIntegerCompareLesss

boolean types are considered (unsigned) integral types.
Using boolean types with GfNumericCast would implicitly cast to a signed
integer in GfIntegerCompareLess, causing Windows to issue C4018 warnings

Author: nvidia-jomiller

pxr/base/gf/numericCast.h

@@ -31,14 +31,27 @@ GfIntegerCompareLess(T t, U u) noexcept
 {
     static_assert(std::is_integral_v<T> && std::is_integral_v<U>);
 
-    if constexpr (std::is_signed_v<T> == std::is_signed_v<U>) {
+    if constexpr (std::is_signed_v<T> && std::is_signed_v<U>) {
         return t < u;
     }
     else if constexpr (std::is_signed_v<T>) {
-        return t < 0 || std::make_unsigned_t<T>(t) < u;
+        if constexpr (std::is_same_v<U, bool>) {
+            return t < 0 || std::make_unsigned_t<T>(t) < std::make_unsigned_t<int>(u);
+        }
+        else {
+            return t < 0 || std::make_unsigned_t<T>(t) < u;
+        }
     }
     else {
-        return u >= 0 && t < std::make_unsigned_t<U>(u);
+        if constexpr (std::is_same_v<T, bool>) {
+            return u >= 0 && std::make_unsigned_t<int>(t) < std::make_unsigned_t<U>(u);
+        }
+        else if constexpr (std::is_same_v<U, bool>) {
+            return t < std::make_unsigned_t<int>(u);
+        }
+        else {
+            return u >= 0 && t < std::make_unsigned_t<U>(u);
+        }
     }
 }