Skip to content

Commit

Permalink
[SYCL] Remove the implicitly passed -ze-take-global-address IGC option (
Browse files Browse the repository at this point in the history
intel#12509)

The -ze-take-global-address IGC option was required for IGC to correctly
record addresses of device globals. This should be on automatically in
newer IGC versions however, so this commit removes the option.

Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
  • Loading branch information
steffenlarsen authored Feb 14, 2024
1 parent e5da67b commit 7e414a9
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions sycl/source/detail/program_manager/program_manager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -380,13 +380,6 @@ static void appendCompileOptionsFromImage(std::string &CompileOpts,
return Dev.is_gpu() &&
Dev.get_info<info::device::vendor_id>() == 0x8086;
});
if (IsIntelGPU && Img.getDeviceGlobals().size() != 0) {
// If the image has device globals we need to add the
// -ze-take-global-address option to tell IGC to record addresses of these.
if (!CompileOpts.empty())
CompileOpts += " ";
CompileOpts += "-ze-take-global-address";
}
if (!CompileOptsEnv) {
static const char *TargetCompileFast = "-ftarget-compile-fast";
if (auto Pos = CompileOpts.find(TargetCompileFast);
Expand Down

0 comments on commit 7e414a9

Please sign in to comment.