microsoft / ADBench

Benchmarking various AD tools.
MIT License
107 stars 39 forks source link

Add missing include in finite.h #210

Closed tomsmeding closed 3 years ago

tomsmeding commented 3 years ago

On my machine, the C++ code failed to build because of the missing include in this PR. Apparently, std::numeric_limits was indirectly included/defined in one of the other header files on older GCC versions; I'm running GCC 11.1.0.

toelli-msft commented 3 years ago

Thanks! @dcrc2 or @awf can you check this for me? (i.e. provide the C++ expertise to confirm this is a sensible thing to do?)