Revert back changes to default constructor

Author: lbushi25

clang/lib/Sema/SemaSYCL.cpp

@@ -3950,23 +3950,13 @@ class SyclKernelBodyCreator : public SyclKernelFieldHandler {
   }
 
   // Default inits the type, then calls the init-method in the body.
-  // A type may not have a public default constructor as per its spec so
-  // typically if this is the case the default constructor will be private and
-  // in such cases we must manually override the access specifier from private
-  // to public just for the duration of this default initialization.
   bool handleSpecialType(FieldDecl *FD, QualType Ty) {
-    const auto *RecordDecl = Ty->getAsCXXRecordDecl();
-    AccessSpecifier DefaultConstructorAccess;
-    auto DefaultConstructor =
-        std::find_if(RecordDecl->ctor_begin(), RecordDecl->ctor_end(),
-                     [](auto it) { return it->isDefaultConstructor(); });
-    DefaultConstructorAccess = DefaultConstructor->getAccess();
-    DefaultConstructor->setAccess(AS_public);
     addFieldInit(FD, Ty, std::nullopt,
                  InitializationKind::CreateDefault(KernelCallerSrcLoc));
-    DefaultConstructor->setAccess(DefaultConstructorAccess);
+
     addFieldMemberExpr(FD, Ty);
 
+    const auto *RecordDecl = Ty->getAsCXXRecordDecl();
     createSpecialMethodCall(RecordDecl, getInitMethodName(), BodyStmts);
     CXXMethodDecl *FinalizeMethod =
         getMethodByName(RecordDecl, FinalizeMethodName);

sycl/include/sycl/ext/oneapi/experimental/work_group_memory.hpp

@@ -49,14 +49,15 @@ class __SYCL_SPECIAL_CLASS __SYCL_TYPE(work_group_memory) work_group_memory
   using decoratedPtr = typename sycl::detail::DecoratedType<
       value_type, access::address_space::local_space>::type *;
 
+public:
   // Frontend requires special types to have a default constructor in order to
   // have a uniform way of initializing an object of special type to then call
-  // the __init method on it. This is purely an implementation detail and not
-  // part of the spec.
+  // the __init method on it. This is currently not part of the spec.
+  // TODO: Remove this once https://github.com/intel/llvm/issues/16061 is
+  // closed.
   work_group_memory() = default;
 
-public:
-  work_group_memory(const indeterminate_t &) {};
+  work_group_memory(const indeterminate_t &){};
   work_group_memory(const work_group_memory &rhs) = default;
   work_group_memory &operator=(const work_group_memory &rhs) = default;
   template <typename T = DataT,