The bazel build of lce_benchmark_model works fine, but when using the cmake version, none of our ops are registered: RegisterSelectedOps is not called at all. This seems to be caused by the fact that the registration function is marked as 'weak', and then it depends on the order of linking which function wins.
There wasn't supposed to be a weak attribute here at all: the weak version of the symbol is in the TF library, meant to be overridden by binaries such as this one.
How Has This Been Tested?
Both the cmake and bazel builds are now working as expected.
What do these changes do?
The bazel build of
lce_benchmark_modelworks fine, but when using the cmake version, none of our ops are registered:RegisterSelectedOpsis not called at all. This seems to be caused by the fact that the registration function is marked as 'weak', and then it depends on the order of linking which function wins.There wasn't supposed to be a weak attribute here at all: the weak version of the symbol is in the TF library, meant to be overridden by binaries such as this one.
How Has This Been Tested?
Both the cmake and bazel builds are now working as expected.