[OPENMP] allow static local variable on data-sharing attribute clause

[kkwli0]

Summary

This patch is to fix the problem of specifying static local variables on the data-sharing attribute clause. Currently, clang flags it as errors.

int main() {
 static int x; 
  int i; 
#pragma omp parallel for reduction(+:x) 
  for (i=0; i<10; i++) { 
    x = 1; 
  } 
}

[ABataev]

Test?

[kkwli0]

Updated unit tests.

[ABataev]

Codegen test? Also for variables, used in chain of different constructs like OpenMP pragmas, blocks, lambdas.

[kkwli0]

Update codegen test cases.

[kkwli0]

No code change other than resolving merge conflict with trunk.

[ABataev]

LG

[kkwli0]

Committed revision 247715.

lib/Sema/SemaOpenMP.cpp

if (!isParallelOrTaskRegion(Kind)) {
 
     // OpenMP [2.9.1.1, Data-sharing Attribute Rules for Variables Referenced
     // in a Construct, C/C++, predetermined, p.4]
     //  Static data members are shared.
     // OpenMP [2.9.1.1, Data-sharing Attribute Rules for Variables Referenced
     // in a Construct, C/C++, predetermined, p.7]
     //  Variables with static storage duration that are declared in a scope
     //  inside the construct are shared.
-    if (D->isStaticDataMember() || D->isStaticLocal()) {
+    if (D->isStaticDataMember()) {
       DSAVarData DVarTemp =
           hasDSA(D, isOpenMPPrivate, MatchesAlways(), FromParent);
       if (DVarTemp.CKind != OMPC_unknown && DVarTemp.RefExpr)
         return DVar;
 
       DVar.CKind = OMPC_shared;
       return DVar;
     }