Respond to comments

Author: Hugh Delaney

sycl/doc/EnvironmentVariables.md

@@ -204,16 +204,18 @@ SYCL runtime this can change the stride offset of different dimensions. Range
 rounding will only be used if the SYCL runtime X dimension exceeds 32, which
 is a magic number chosen by the SYCL runtime.
 
-Generation of range rounded kernels can be disabled by defining the macro
-`__SYCL_DISABLE_PARALLEL_FOR_RANGE_ROUNDING__` at compile time. The macro must
-be defined before `#include <sycl/sycl.hpp>`.
+Generation of range rounded kernels can be disabled by using the compiler flag
+`-fsycl-disable-range-rounding`.
 
 ### Range Rounding Environment Variables
 
 | Environment variable | Values | Description |
 | -------------------- | ------ | ----------- |
 | `SYCL_DISABLE_PARALLEL_FOR_RANGE_ROUNDING` | Any(\*) | Disables automatic rounding-up of `parallel_for` invocation ranges. |
 | `SYCL_PARALLEL_FOR_RANGE_ROUNDING_TRACE`   | Any(\*) | Enables tracing of `parallel_for` invocations with rounded-up ranges. |
+| `SYCL_PARALLEL_FOR_RANGE_ROUNDING_PARAMS`  | `MinFactorX:GoodFactor:MinRangeX` | `MinFactorX`: The minimum range that the rounded range should be a multiple of (Default 16)  |
+|  |  | `GoodFactor`: The preferred range that the rounded range be a multiple of (Default 32)  |
+|  |  | `MinRangeX`: The minimum X dimension of the range such that range rounding is activated (Default 1024) |
 
 
 ## Controlling DPC++ Level Zero Plugin

sycl/doc/PreprocessorMacros.md

@@ -48,13 +48,6 @@ This file describes macros that have effect on SYCL compiler and run-time.
   support for `assert()` via `aspect::ext_oneapi_native_assert`.
   This macro is undefined by default.
 
-- **__SYCL_DISABLE_PARALLEL_FOR_RANGE_ROUNDING__**
-
-  Defining this macro disables the automatic generation of a range rounded
-  kernel for `sycl::range` kernels. The macro must be defined before
-  `#include <sycl/sycl.hpp>`. Using this macro may reduce device binary size if
-  `sycl::range` kernels are used.
-
 - **SYCL2020_CONFORMANT_APIS (deprecated)**
   This macro is used to comply with the SYCL 2020 specification, as some of the current 
   implementations may be widespread and not conform to it.

sycl/include/sycl/handler.hpp

@@ -1139,7 +1139,7 @@ class __SYCL_EXPORT handler {
     // Range should be a multiple of this for improved performance.
     size_t GoodFactor = 32;
     // Range should be at least this to make rounding worthwhile.
-    size_t MinRangeX = 32;
+    size_t MinRangeX = 1024;
 
     // Check if rounding parameters have been set through environment:
     // SYCL_PARALLEL_FOR_RANGE_ROUNDING_PARAMS=MinRound:PreferredRound:MinRange