[PATCH] D157204: [MLIR][Math] Add support for f16 in the expansion of math.roundeven

[Nathaniel McVicar via Phabricator via llvm-commits]

NathanielMcVicar added a comment.

This appears to have broken the windows bot: https://lab.llvm.org/buildbot/#/builders/13/builds/39032/steps/6/logs/stdio

  638.315 [1228/28/2875] Building CXX object tools\mlir\lib\ExecutionEngine\CMakeFiles\mlir_c_runner_utils.dir\SparseTensorRuntime.cpp.obj
  FAILED: tools/mlir/lib/ExecutionEngine/CMakeFiles/mlir_c_runner_utils.dir/SparseTensorRuntime.cpp.obj 
  C:\buildbot\mlir-x64-windows-ninja\llvm-project\mlir\include\mlir/ExecutionEngine/Float16bits.h(51): error C2220: the following warning is treated as an error
  C:\buildbot\mlir-x64-windows-ninja\llvm-project\mlir\include\mlir/ExecutionEngine/Float16bits.h(51): warning C4273: 'printF16': inconsistent dll linkage
  C:\buildbot\mlir-x64-windows-ninja\llvm-project\mlir\include\mlir/ExecutionEngine/CRunnerUtils.h(472): note: see previous definition of 'printF16'
  C:\buildbot\mlir-x64-windows-ninja\llvm-project\mlir\include\mlir/ExecutionEngine/Float16bits.h(52): warning C4273: 'printBF16': inconsistent dll linkage
  C:\buildbot\mlir-x64-windows-ninja\llvm-project\mlir\include\mlir/ExecutionEngine/CRunnerUtils.h(473): note: see previous definition of 'printBF16'


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D157204/new/

https://reviews.llvm.org/D157204