Address comments

Author: romanovvlad

sycl/ReleaseNotes.md

@@ -1,6 +1,10 @@
 # May'21 release notes
 
 ## New features
+ - [ESIMD] Allowed ESIMD and regular SYCL kernels to coexist in the same
+   translation unit and in the same program. The `-fsycl-explicit-simd` option
+   is no longer required for compiling ESIMD code and was deprecated. DPCPP RT
+   implicitly appends `-vc-codegen` compile option for ESIMD images.
  - [ESIMD] Added indirect read and write methods to ESIMD class [8208427]
  - Provided `sycl::ONEAPI::has_known_identity` type trait to determine if
    reduction interface supports user-defined type at compile-time [0c7bd24]
@@ -22,8 +26,6 @@
  - Implemented zero argument version of `sycl::buffer::reinterpret()` for
    SYCL 2020 [c0c3c80]
  - Implemented [Matrix Programming Extension for DPC++](https://github.com/intel/llvm/blob/49b6749ea9175ae250b718c04d71af4ccfecc06c/sycl/doc/extensions/Matrix/dpcpp-joint-matrix.asciidoc) [35db973]
- - Added support SYCL2020 style interoperability API for OpenCL backend
-   [12e8549] [c2f211a]
  - Added support for
    [SYCL_INTEL_local_memory extension](doc/extensions/LocalMemory/SYCL_INTEL_local_memory.asciidoc)
    [5a66fcb] [9a734f6]
@@ -32,11 +34,11 @@
 
 ## Improvements
 ### SYCL Compiler
- - Added support for math builtins: `fmax`, `fmin`, `isinf`, `isfinite`,
+ - Added support for math built-ins: `fmax`, `fmin`, `isinf`, `isfinite`,
    `isnormal`, `fpclassify` [1040b94]
  - The FPGA initiation interval attribute spelling `[[intel::ii]]` is
    deprecated. The new spelling is `[[intel::initiation_interval]]`. In
-   addition `[[intel::initiation_interval]]` may now be used as a function
+   addition, `[[intel::initiation_interval]]` may now be used as a function
    attribute, formerly its use was limited to statement attribute [b04e6a0]
  - Added support for function attribute `[[intel::disable_loop_pipelining]]`
    and `[[intel::max_concurrency(n)]]` [7324b3e]
@@ -61,24 +63,23 @@
  - Provided facility for user to control execution range rounding [f6ac45f]
  - Ensured correct access mode in `sycl::handler::copy()` method [b489479]
  - Disallowed for atomic accessors in `sycl::handler::copy()` method [14437db]
- - Implicitly added `-vc-codegen` compile option for ESIMD images [798b4c5]
  - Provided move-assignability of `usm_allocator` class [05a805e]
- - Improved performance when using `COPY_HOST_PTR` at devices without host
-   unified memory [ad8c9d1]
+ -  Improved performance of copying data during native memory object creation
+    on devices without host unified memory [ad8c9d1]
  - [ESIMD] Added implicit set up of fence before barrier as required by hardware
    [692228c]
  - Allowed for using of interoperability program constructor with multi-device
    context [c7f7674]
  - Allowed trace of Level Zero calls only with `SYCL_PI_TRACE=-1` [ea73219]
- - Added throw of `feature_not_supported` when upon attempt to create program
-   using `create_program_with_source` with Level Zero or CUDA [ba77e3a]
+ - Added throw of `feature_not_supported` when when upon attempt to create
+   program using `create_program_with_source` with Level Zero or CUDA [ba77e3a]
  - Added support for `inline` `cl` namespace in debugger [8e441d4]
- - Added support build with GCC 7 [d8fea22]
- - Added in-memory caching of programs which are built with custom build options
+ - Added support for build with GCC 7 [d8fea22]
+ - Added in-memory caching of programs built with custom build options
    [86b0e8d] [e152b0d]
  - Improved range rounding heuristics [7efb692]
  - Added `get_backend` methods to SYCL classes [ee7e99f]
- - Added `sycl::sub_group::load` and `sycl::sub_group::store` versions which
+ - Added `sycl::sub_group::load` and `sycl::sub_group::store` versions that
    take raw pointers [248f550]
  - Enabled caching of devices in `sycl::device` interoperability constructors
    [d3aeb4a]
@@ -113,15 +114,15 @@
 
 ## Bug fixes
 ### SYCL Compiler
- - Suppressed link time warning on Windows which incorrectly diagnoses
+ - Suppressed link time warning on Windows that incorrectly diagnosed
    conflicting section names while linking device binaries [8e6a3ec]
  - Disabled code coverage for device compilations [12a0b11]
  - Fixed an issue when unbundling a fat static archive and targeting non-FPGA
    device [90c79c7]
  - Addressed inconsistencies when performing compilations by using the target
    triple for FPGA (`spir64_fpga-unknown-unknown-sycldevice`) vs using
    `-fintelfpga` [c9a65fc]
- - Fixed generation the output report folder when performing FPGA AOT
+ - Fixed generation of the output report folder when performing FPGA AOT
    compilations from a previously generated AOCR archive [eab4791]
  - Addressed issues dealing with improper settings when performing
    preprocessing when offloading is enabled [d03de03]
@@ -137,7 +138,7 @@
    `-fsycl-device-only` [3d2225a]
 ### SYCL Library
  - Fixed race-condition happening on application exit [8eb00d7] [c9c1de9]
- - Fixed faulty behaviour which happened when accessing a buffer in different
+ - Fixed faulty behaviour that happened when accessing a buffer in different
    contexts using `discard_*` access mode [f75b439]
  - Fixed support for `SYCL_PROGRAM_LINK_OPTIONS` and
    `SYCL_PROGRAM_COMPILE_OPTIONS` environment variables when compiling/linking
@@ -151,14 +152,14 @@
    `sycl::buffer::set_final_data()` method [084d83a, 2a751bd]
  - Fixed support for `long long` in `sycl::vec::convert()` on Windows [5b49cd3]
  - Aligned local and image accessor with specification by allowing for property
-   list in its constructor [88fab25]
+   list in their constructor [88fab25]
  - Fixed support for offset in `parallel_for` for host device [1958715]
  - Added missing constructors for `sycl::buffer` class [bdfad9e]
  - Fixed coordinate conversion for `sampler` class on host device [cd6529f]
- - Fixed for support of local accessor in debugger [fdacb75]
+ - Fixed support for local accessors in debugger [fdacb75]
  - Fixed dropping of kernel attributes when execution range rounding is used
    [496f9a0] [677a7ea]
- - Added support for interoperability tasks which use `get_mem()` methods with
+ - Added support for interoperability tasks that use `get_mem()` methods with
    Level Zero plugin [149f08d]
  - Fixed sub-device caching in the Level Zero plugin [0b18b49]
  - Fixed `get_native` methods to retain reference counter in case of OpenCL
@@ -167,14 +168,14 @@
    they have been signaled [2a76b2a]
  - Resolved a pinned host memory specific performance regression on CUDA that
    was introduced with the host unified behavior dependent logic [3be63ab]
- - Fixed illegal accesses which could happen when an application which uses
-   host tasks exits without waiting for host tasks completion [552a521]
+ - Fixed illegal accesses that could happen when an application that uses host
+   tasks exits without waiting for host tasks completion [552a521]
  - Fixed `sycl::event::get_info` queries that were working incorrectly when
    called on event without an encapsulated native handle [5d5a792]
  - Fixed compilation error with using multidimensional subscript for
    `sycl::accessor` with atomic access mode [0bfd34e]
- - Fixed a crash which happened when an accessor which is passed to the
-   reduction is just after being passed to reduction [b80f13e]
+ -  Fixed a crash that happened when an accessor passed to a reduction was
+    destroyed immediately after [b80f13e]
  - Fixed `sycl::device::get_info` with `sycl::info::device::max_mem_alloc_size`
    which was returning incorrect value in case of Level Zero backend [8dbaa53]
 
@@ -202,8 +203,8 @@
    versions of C++ RT used on app and sycl[d].dll sides.
  - The format of the object files produced by the compiler can change between
    versions. The workaround is to rebuild the application.
- - Using `cl::sycl::program` API to refer to a kernel defined in another
-   translation unit leads to undefined behavior
+ - Using `sycl::program`/`sycl::kernel_bundle` API to refer to a kernel defined
+   in another translation unit leads to undefined behavior
  - Linkage errors with the following message:
    `error LNK2005: "bool const std::_Is_integral<bool>" (??$_Is_integral@_N@std@@3_NB) already defined`
    can happen when a SYCL application is built using MS Visual Studio 2019