Apply comments

Added an example of composite constant descriptor. Fixed a few typos.
Applied clang-format

Author: AlexeySachkov

sycl/include/CL/sycl/detail/pi.hpp

@@ -359,20 +359,29 @@ class DeviceBinaryImage {
     return Format;
   }
 
-  /// Gets the iterator range over scalar specialization constants in this this
+  /// Gets the iterator range over scalar specialization constants in this
   /// binary image. For each property pointed to by an iterator within the
-  /// range, the name of the property is the specializaion constant symbolic ID
+  /// range, the name of the property is the specialization constant symbolic ID
   /// and the value is 32-bit unsigned integer ID.
   const PropertyRange &getScalarSpecConstants() const {
     return ScalarSpecConstIDMap;
   }
   /// Gets the iterator range over composite specialization constants in this
-  /// this binary image. For each property pointed to by an iterator within the
-  /// range, the name of the property is the specializaion constant symbolic ID
+  /// binary image. For each property pointed to by an iterator within the
+  /// range, the name of the property is the specialization constant symbolic ID
   /// and the value is a list of tuples of 32-bit unsigned integer values, which
-  /// encode scalar specialization constants, that form a composite one.
+  /// encode scalar specialization constants, that form the composite one.
   /// Each tuple consist of ID of scalar specialization constant, its location
   /// within a composite (offset in bytes from the beginning) and its size.
+  /// For example, for the following structure:
+  /// struct A { int a; float b; };
+  /// struct POD { A a[2]; int b; };
+  /// List of tuples will look like:
+  /// { ID0, 0, 4 },  // .a[0].a
+  //  { ID1, 4, 4 },  // .a[0].b
+  //  { ID2, 8, 4 },  // .a[1].a
+  //  { ID3, 12, 4 }, // .a[1].b
+  //  { ID4, 16, 4 }, // .b
   const PropertyRange &getCompositeSpecConstants() const {
     return CompositeSpecConstIDMap;
   }

sycl/test/composite-spec-const.cpp

@@ -74,11 +74,8 @@ int main(int argc, char **argv) {
 
     q.submit([&](cl::sycl::handler &cgh) {
       auto acci = bufi.get_access<cl::sycl::access::mode::write>(cgh);
-      cgh.single_task<Test>(
-          program.get_kernel<Test>(),
-          [=]() {
-            acci[0] = pod.get();
-          });
+      cgh.single_task<Test>(program.get_kernel<Test>(),
+                            [=]() { acci[0] = pod.get(); });
     });
   } catch (cl::sycl::exception &e) {
     std::cout << "*** Exception caught: " << e.what() << "\n";