Skip to content

PR #40331: [XLA:GPU][oneAPI] Fix build failure in IntelGpuCompiler#40371

Open
copybara-service[bot] wants to merge 1 commit intomainfrom
test_894161828
Open

PR #40331: [XLA:GPU][oneAPI] Fix build failure in IntelGpuCompiler#40371
copybara-service[bot] wants to merge 1 commit intomainfrom
test_894161828

Conversation

@copybara-service
Copy link
Copy Markdown

PR #40331: [XLA:GPU][oneAPI] Fix build failure in IntelGpuCompiler

Imported from GitHub PR #40331

00b5900 commit in XLA causes the following failure in oneAPI GPU CI build:

./xla/service/gpu/intel_gpu_compiler.h:57:71: error: non-virtual member function marked 'override' hides virtual member function
   57 |       const CompileOptions& options, std::optional<int> shard_number) override;
      |                                                                       ^
./xla/service/gpu/gpu_compiler.h:266:48: note: hidden overloaded virtual function 'xla::gpu::GpuCompiler::CompileTargetBinary' declared here: different number of parameters (6 vs 7)
  266 |   virtual absl::StatusOr<BackendCompileResult> CompileTargetBinary(

This PR fixes the failure by updating the function signature of CompileTargetBinary to match the base class.
Copybara import of the project:

--
63fd38e by Bhavani Subramanian bhavani1.subramanian@intel.com:

Fix build failure in IntelGpuCompiler::CompileTargetBinary

Merging this change closes #40331

FUTURE_COPYBARA_INTEGRATE_REVIEW=#40331 from Intel-tensorflow:bhavanis/oneapi-build-fix 63fd38e

Imported from GitHub PR #40331

00b5900 commit in XLA causes the following failure in oneAPI GPU CI build:
```
./xla/service/gpu/intel_gpu_compiler.h:57:71: error: non-virtual member function marked 'override' hides virtual member function
   57 |       const CompileOptions& options, std::optional<int> shard_number) override;
      |                                                                       ^
./xla/service/gpu/gpu_compiler.h:266:48: note: hidden overloaded virtual function 'xla::gpu::GpuCompiler::CompileTargetBinary' declared here: different number of parameters (6 vs 7)
  266 |   virtual absl::StatusOr<BackendCompileResult> CompileTargetBinary(
```
This PR fixes the failure by updating the function signature of `CompileTargetBinary` to match the base class.
Copybara import of the project:

--
63fd38e by Bhavani Subramanian <bhavani1.subramanian@intel.com>:

Fix build failure in IntelGpuCompiler::CompileTargetBinary

Merging this change closes #40331

FUTURE_COPYBARA_INTEGRATE_REVIEW=#40331 from Intel-tensorflow:bhavanis/oneapi-build-fix 63fd38e
PiperOrigin-RevId: 894161828
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant