-
Notifications
You must be signed in to change notification settings - Fork 487
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
4 changed files
with
36 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
upstream CI will fail without this | ||
diff --git a/xla/service/cpu/runtime_fp16.h b/xla/service/cpu/runtime_fp16.h | ||
index 3f7af5197..ce4491c5d 100644 | ||
--- a/xla/service/cpu/runtime_fp16.h | ||
+++ b/xla/service/cpu/runtime_fp16.h | ||
@@ -18,12 +18,7 @@ limitations under the License. | ||
|
||
#include <stdint.h> | ||
|
||
-// _Float16 always gets us the correct ABI type, so use that if available. | ||
-// AArch64 GCC defines __FLT16_MANT_DIG__ even when _Float16 is not available. | ||
-#if defined(__FLT16_MANT_DIG__) && \ | ||
- (defined(__clang__) || !(defined(__GNUC__) && defined(__aarch64__))) | ||
-using XlaF16ABIType = _Float16; | ||
-#elif defined(__x86_64__) | ||
+#if defined(__x86_64__) | ||
// Older versions of Clang don't have _Float16. Since both float and _Float16 | ||
// are passed in the same register we can use the wider type and careful casting | ||
// to conform to x86_64 psABI. This only works with the assumption that we're |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
diff --git a/xla/service/gpu/gpu_executable.cc b/xla/service/gpu/gpu_executable.cc | ||
index 242961dd1..787275868 100644 | ||
--- a/xla/service/gpu/gpu_executable.cc | ||
+++ b/xla/service/gpu/gpu_executable.cc | ||
@@ -563,8 +563,7 @@ StatusOr<ExecutionOutput> GpuExecutable::ExecuteAsyncOnStreamImpl( | ||
} | ||
|
||
// Force synchronous execution if the allocator requires it. | ||
- const bool block_host_until_done = | ||
- !memory_allocator->AllowsAsynchronousDeallocation(); | ||
+ const bool block_host_until_done = true; | ||
|
||
|
||
// Lock the GPU with a shared lock so that we don't interfere with autotuning |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters