search

lukeiwanski / tensorflow

OpenCL support for TensorFlow via SYCL
Apache License 2.0
65 stars 14 forks source link

[OpenCL] Fixes SYCL profiler tests #141

Closed jwlawson closed 7 years ago

jwlawson commented 7 years ago

The profiler relies heavily on the canonical device being listed in the TFProf nodes, which is only set for those devices which return True from CountAsCPUTime, so we need this to return True for SYCL device nodes too. The check for whether the node will run on an Accelerator comes from IsPlacedOnAccelerator.

jwlawson commented 7 years ago

@lukeiwanski Your patch 10574d8 needs to be reverted as well (its not in dev/integration yet, so I can't revert it in this PR).