Skip to content

Fix regression in XLA in TensorFlow 1.13 RC0, RC1#25769

Merged
aselle merged 1 commit into
tensorflow:r1.13from
aselle:cp-20190214b
Feb 15, 2019
Merged

Fix regression in XLA in TensorFlow 1.13 RC0, RC1#25769
aselle merged 1 commit into
tensorflow:r1.13from
aselle:cp-20190214b

Conversation

@aselle

@aselle aselle commented Feb 15, 2019

Copy link
Copy Markdown
Contributor

Only convert BackpropFilterConv to depthwise convolution if format is… … NHWC.

The logic for the conversion has this assumption. This CL makes sure that this
assumption holds, and adds tests for NCHW format.

Fixes #25734

PiperOrigin-RevId: 225534508

… NHWC.

The logic for the conversion has this assumption. This CL makes sure that this
assumption holds, and adds tests for NCHW format.

PiperOrigin-RevId: 225534508
@ymodak ymodak added the prtype:bugfix PR to fix a bug label Feb 15, 2019
@aselle aselle merged commit c865ec5 into tensorflow:r1.13 Feb 15, 2019
copybara-service Bot pushed a commit that referenced this pull request Apr 30, 2025
…node workaround

Imported from GitHub PR openxla/xla#25769

CUDA release r570/CUDA 12.8 has fixed the bug that cuda will hang when cuda-graph's conditional node has an empty body graph.  Remove the workaround in XLA, tests have been covered by GpuCommandBufferTest.ConditionalCaseEmptyGraph

Copybara import of the project:

--
355457198b24b1053ec79eacea13176e0751a5f8 by Shawn Wang <[email protected]>:

remove cuda graph emtpy node inside conditional node workaround

Merging this change closes #25769

FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#25769 from shawnwang18:shawnw/remove_empty_graph_workaround 355457198b24b1053ec79eacea13176e0751a5f8
PiperOrigin-RevId: 752342132
copybara-service Bot pushed a commit that referenced this pull request Apr 30, 2025
…node workaround

Imported from GitHub PR openxla/xla#25769

CUDA release r570/CUDA 12.8 has fixed the bug that cuda will hang when cuda-graph's conditional node has an empty body graph.  Remove the workaround in XLA, tests have been covered by GpuCommandBufferTest.ConditionalCaseEmptyGraph

Copybara import of the project:

--
355457198b24b1053ec79eacea13176e0751a5f8 by Shawn Wang <[email protected]>:

remove cuda graph emtpy node inside conditional node workaround

Merging this change closes #25769

PiperOrigin-RevId: 753069210
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants