[CUDA] remove duplicate condition

[yaxunl]

LGTM. Thanks.

[VitalyR]

@yaxunl Hi! Could you commit this on my behalf? I read the documentation and it seems the appropriate way to commit changes for a newcomer not having commit access like me. My name is "VitalyR" and my email address is "[e-mail]". Thanks!

[yaxunl]

In D130168#3667283, @VitalyR wrote:

@yaxunl Hi! Could you commit this on my behalf? I read the documentation and it seems the appropriate way to commit changes for a newcomer not having commit access like me. My name is "VitalyR" and my email address is "[e-mail]". Thanks!

sure

[VitalyR]

Thanks!

clang/lib/Sema/SemaCUDA.cpp

 
 // TODO: `__constant__` memory may be a limited resource for certain targets.
 // A safeguard may be needed at the end of compilation pipeline if
 // `__constant__` memory usage goes beyond limit.
 void Sema::MaybeAddCUDAConstantAttr(VarDecl *VD) {
   // Do not promote dependent variables since the cotr/dtor/initializer are
   // not determined. Do it after instantiation.
   if (getLangOpts().CUDAIsDevice && !VD->hasAttr<CUDAConstantAttr>() &&
-      !VD->hasAttr<CUDAConstantAttr>() && !VD->hasAttr<CUDASharedAttr>() &&
+      !VD->hasAttr<CUDASharedAttr>() &&
       (VD->isFileVarDecl() || VD->isStaticDataMember()) &&
       !IsDependentVar(VD) &&
       ((VD->isConstexpr() || VD->getType().isConstQualified()) &&
        HasAllowedCUDADeviceStaticInitializer(*this, VD,
                                              CICK_DeviceOrConstant))) {
     VD->addAttr(CUDAConstantAttr::CreateImplicit(getASTContext()));
   }
 }