tree 4bdac396ca186a374de6c86133ab1dd6dd7202b2
parent 39a590dfb66db999ace422be51513c0f04939f09
author Antonio Sanchez <cantonios@google.com> 1614381900 -0800
committer David Tellenbach <david.tellenbach@me.com> 1614461526 +0000

Make half/bfloat16 constructor take inputs by value, fix powerpc test.

Since `numeric_limits<half>::max_exponent` is a static inline constant,
it cannot be directly passed by reference. This triggers a linker error
in recent versions of `g++-powerpc64le`.

Changing `half` to take inputs by value fixes this.  Wrapping
`max_exponent` with `int(...)` to make an addressable integer also fixes this
and may help with other custom `Scalar` types down-the-road.

Also eliminated some compile warnings for powerpc.