Improve comment and style

Author: intel-liudean

include/acl.h

@@ -515,15 +515,16 @@ typedef enum {
 // Enum values here also need to match the SPIRV spec for device
 // global in the above link for acl_device_global_host_access_t.
 
-
 // Definition of device global.
 struct acl_device_global_mem_def_t {
   uint64_t address;
   uint32_t size;
   acl_device_global_host_access_t host_access;
-  bool can_skip_programming; // every device global has the same value
+  // every device global has the same value for can_skip_programming
+  bool can_skip_programming;
   bool implement_in_csr;
-  bool reset_on_reuse; // every device global has the same value
+  // every device global has the same value for reset_on_reuse
+  bool reset_on_reuse;
 };
 
 // Part of acl_device_def_t where members are populated from the information

src/acl_auto_configure.cpp

@@ -530,8 +530,8 @@ static bool read_device_global_mem_defs(
     }
     bool can_skip_programming = false;
     if (result && counters.back() > 0) {
-      result =
-          read_bool_counters(config_str, curr_pos, can_skip_programming, counters);
+      result = read_bool_counters(config_str, curr_pos, can_skip_programming,
+                                  counters);
     }
     bool implement_in_csr = false;
     if (result && counters.back() > 0) {
@@ -545,9 +545,12 @@ static bool read_device_global_mem_defs(
     }
 
     acl_device_global_mem_def_t dev_global_def = {
-        dev_global_addr, dev_global_size,
+        dev_global_addr,
+        dev_global_size,
         static_cast<acl_device_global_host_access_t>(host_access),
-        can_skip_programming, implement_in_csr, reset_on_reuse};
+        can_skip_programming,
+        implement_in_csr,
+        reset_on_reuse};
     bool ok =
         device_global_mem_defs.insert({device_global_name, dev_global_def})
             .second;

test/acl_auto_configure_test.cpp

@@ -99,7 +99,7 @@ TEST(auto_configure, simple) {
 
 // Device global autodiscovery entries
 #define NUM_DEV_GLOBAL " 2"
-#define NUM_DEV_GLOBAL_FIELD " 7" 
+#define NUM_DEV_GLOBAL_FIELD " 7"
 // The 7 fields are dev_globa_name, address, size, host_access,
 //  can_skip_programming, implement_in_csr, reset_on_reuse
 #define DEV_GLOBAL_1 " kernel15_dev_global 0x1000 2048 3 0 0 0"
@@ -286,15 +286,13 @@ TEST(auto_configure, simple) {
   CHECK_EQUAL(2048, kernel15_dev_global->second.size);
   CHECK_EQUAL(ACL_DEVICE_GLOBAL_HOST_ACCESS_NONE,
               kernel15_dev_global->second.host_access);
-  CHECK_EQUAL(false,
-              kernel15_dev_global->second.can_skip_programming);
+  CHECK_EQUAL(false, kernel15_dev_global->second.can_skip_programming);
   CHECK_EQUAL(false, kernel15_dev_global->second.implement_in_csr);
   CHECK_EQUAL(2048, kernel15_dev_global2->second.address);
   CHECK_EQUAL(1024, kernel15_dev_global2->second.size);
   CHECK_EQUAL(ACL_DEVICE_GLOBAL_HOST_ACCESS_WRITE_ONLY,
               kernel15_dev_global2->second.host_access);
-  CHECK_EQUAL(false,
-              kernel15_dev_global2->second.can_skip_programming);
+  CHECK_EQUAL(false, kernel15_dev_global2->second.can_skip_programming);
   CHECK_EQUAL(true, kernel15_dev_global2->second.implement_in_csr);
 
   // Check a second parsing.