You never actually call DeclarationMarkedOpenMPThreadPrivate in the first 
place.  The right place to do that is at the point when you add the attribute.

You also need to test that this works.  The basic idea is that the first PCH 
file should declare the variable, the second should declare it to be 
thread-private, and the third uses it (in some way that verifies that it was 
marked thread-private; preferably using a Sema check, but an IRGen test is also 
okay).  You can look at test/PCH/chain-categories.m and 
test/PCH/chain-implicit-definition.cpp for examples of how to do that.  (The 
test should go in test/PCH, not test/OpenMP.)

http://reviews.llvm.org/D4002



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to