Addressing review feedbacks
This commit is contained in:
parent
98e4579b39
commit
b6171a4eb3
@ -13,8 +13,8 @@ See the License for the specific language governing permissions and
|
|||||||
limitations under the License.
|
limitations under the License.
|
||||||
==============================================================================*/
|
==============================================================================*/
|
||||||
|
|
||||||
#ifndef TENSORFLOW_COMPILER_XLA_SERVICE_GPU_CONV_ALGORITHM_PICKER_H_
|
#ifndef TENSORFLOW_COMPILER_XLA_SERVICE_GPU_GPU_CONV_ALGORITHM_PICKER_H_
|
||||||
#define TENSORFLOW_COMPILER_XLA_SERVICE_GPU_CONV_ALGORITHM_PICKER_H_
|
#define TENSORFLOW_COMPILER_XLA_SERVICE_GPU_GPU_CONV_ALGORITHM_PICKER_H_
|
||||||
|
|
||||||
#include "absl/time/time.h"
|
#include "absl/time/time.h"
|
||||||
#include "absl/types/optional.h"
|
#include "absl/types/optional.h"
|
||||||
@ -67,4 +67,4 @@ class GpuConvAlgorithmPicker : public HloModulePass {
|
|||||||
|
|
||||||
} // namespace gpu
|
} // namespace gpu
|
||||||
} // namespace xla
|
} // namespace xla
|
||||||
#endif // TENSORFLOW_COMPILER_XLA_SERVICE_GPU_CONV_ALGORITHM_PICKER_H_
|
#endif // TENSORFLOW_COMPILER_XLA_SERVICE_GPU_GPU_CONV_ALGORITHM_PICKER_H_
|
||||||
|
@ -229,7 +229,8 @@ Status RunGpuConvImpl(const GpuConvParams& params,
|
|||||||
// first call we need to ensure that the AlgorithmConfig::algorithm is
|
// first call we need to ensure that the AlgorithmConfig::algorithm is
|
||||||
// empty. For all subsequent calls, we should use the value retrieved from
|
// empty. For all subsequent calls, we should use the value retrieved from
|
||||||
// the backend_config
|
// the backend_config
|
||||||
if ((options.algo_override.has_value()) &&
|
if ((stream->parent()->platform_kind() == se::PlatformKind::kROCm) &&
|
||||||
|
(options.algo_override.has_value()) &&
|
||||||
(*options.algo_override == se::dnn::AlgorithmDesc())) {
|
(*options.algo_override == se::dnn::AlgorithmDesc())) {
|
||||||
algorithm = AlgorithmConfig();
|
algorithm = AlgorithmConfig();
|
||||||
} else if (options.algo_override.has_value()) {
|
} else if (options.algo_override.has_value()) {
|
||||||
|
Loading…
Reference in New Issue
Block a user