Skip to content

Commit 92945f4

Browse files
authored
[SYCL][NFC] Rename variable (#13208)
After 370aa2a grf_size control values changed to 128 and 256 values instead of values like "small", "large". > 2) Adds two new kernel properties > `sycl::ext::intel::experimental::grf_size` and > `sycl::ext::intel::experimental::grf_size_automatic`, as per the spec. > `grf_size` adds the `sycl-grf-size` metadata with a value of the > template parameter **(`128` or `256`)**. `grf_size_automatic` adds the > `sycl-grf-size` metadata with a value of `0`. and user is expected to specify value like this: syclex::properties kernel_properties{intelex::grf_size<128>}; syclex::properties kernel_properties{intelex::grf_size<256>};
1 parent 8913285 commit 92945f4

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

sycl/source/detail/program_manager/program_manager.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -284,22 +284,22 @@ appendCompileOptionsForGRFSizeProperties(std::string &CompileOpts,
284284
// The mutual exclusivity of these properties should have been checked in
285285
// sycl-post-link.
286286
assert(!RegAllocModeProp || !GRFSizeProp);
287-
bool IsLargeGRF = false;
287+
bool Is256GRF = false;
288288
bool IsAutoGRF = false;
289289
if (RegAllocModeProp) {
290290
uint32_t RegAllocModePropVal =
291291
DeviceBinaryProperty(RegAllocModeProp).asUint32();
292-
IsLargeGRF = RegAllocModePropVal ==
293-
static_cast<uint32_t>(register_alloc_mode_enum::large);
292+
Is256GRF = RegAllocModePropVal ==
293+
static_cast<uint32_t>(register_alloc_mode_enum::large);
294294
IsAutoGRF = RegAllocModePropVal ==
295295
static_cast<uint32_t>(register_alloc_mode_enum::automatic);
296296
} else {
297297
assert(GRFSizeProp);
298298
uint32_t GRFSizePropVal = DeviceBinaryProperty(GRFSizeProp).asUint32();
299-
IsLargeGRF = GRFSizePropVal == 256;
299+
Is256GRF = GRFSizePropVal == 256;
300300
IsAutoGRF = GRFSizePropVal == 0;
301301
}
302-
if (IsLargeGRF) {
302+
if (Is256GRF) {
303303
if (!CompileOpts.empty())
304304
CompileOpts += " ";
305305
// This option works for both LO AND OCL backends.

0 commit comments

Comments
 (0)