message: Review Comments 2
diff: @@ -126,7 +126,7 @@ inline string GetMklEagerOpName(const string& name) {
  }
  #ifdef ENABLE_INTEL_MKL_BFLOAT16
  -static inline bool HasBfloat16Support(DataType T) {
  +static inline bool CheckBfloat16Support(DataType T) {
  static absl::once_flag cpu_bfloat16_warn_once_flag;
  // Restrict bfloat16 ops to platforms with at least AVX512 support, fall back
  // to Eigen implementation otherwise.
  @@ -159,7 +159,7 @@ static inline bool IsMklLayoutDependentOp(const string& op_name, DataType T) {
  #ifdef ENABLE_INTEL_MKL_BFLOAT16
  // Restrict regular ops to FLOAT and BFLOAT16
  if (kernel.find(kMklLayoutDependentOpLabelPattern) != string::npos) {
  - return (T == DT_FLOAT || (T == DT_BFLOAT16 && HasBfloat16Support(T)));
  + return (T == DT_FLOAT || CheckBfloat16Support(T));
  }
  #else
  // Restrict regular ops to FLOAT
  @@ -217,7 +217,7 @@ static inline bool IsMklNameChangeOp(const string& op_name, DataType T) {
  T == DT_DOUBLE || T == DT_FLOAT);
  #ifdef ENABLE_INTEL_MKL_BFLOAT16
  isTypeAllowed =
  - isTypeAllowed || (T == DT_BFLOAT16 && HasBfloat16Support(T));
  + (isTypeAllowed || CheckBfloat16Support(T));
  #endif
  return isTypeAllowed;
  }
  