comments and cleanup

Author: cperkinsintel

sycl/source/detail/kernel_bundle_impl.hpp

@@ -529,8 +529,7 @@ class kernel_bundle_impl {
     std::string XsFlags = extractXsFlags(BuildOptions);
     auto Res = Adapter->call_nocheck<UrApiKind::urProgramBuildExp>(
         UrProgram, DeviceVec.size(), DeviceVec.data(), XsFlags.c_str());
-    if (Res ==
-        UR_RESULT_ERROR_UNSUPPORTED_FEATURE) { // (Res != UR_RESULT_SUCCESS) {
+    if (Res == UR_RESULT_ERROR_UNSUPPORTED_FEATURE) {
       Res = Adapter->call_nocheck<UrApiKind::urProgramBuild>(
           ContextImpl->getHandleRef(), UrProgram, XsFlags.c_str());
     }

sycl/test-e2e/KernelCompiler/kernel_compiler_sycl.cpp

@@ -182,6 +182,8 @@ void test_build_and_run(bool readingFromCache) {
       syclex::properties{syclex::build_options{flags}, syclex::save_log{&log},
                          syclex::registered_kernel_names{"ff_templated<int>"}});
 
+  // If the kernel was restored from cache, there will not have been
+  // any warning issued by the compilation of the kernel.
   if (!readingFromCache) {
     assert(log.find("warning: 'this_nd_item<1>' is deprecated") !=
            std::string::npos);