Proper treatment of builtin region "Local"

hanm / clang

Mirror of the official Clang GIT repository

Other

2 stars 0 forks source link

Proper treatment of builtin region "Local" #35

Open atzannes opened 11 years ago

atzannes commented 11 years ago

Rename to something more explicit such as ThreadLocal
Disallow region name declarations of the same name
Disallow RPLs containing ThreadLocal AND other RPL elements
Disallow aliasing ThreadLocal with other Regions

vadve commented 11 years ago

On Feb 25, 2013, at 3:20 PM, Alexandros Tzannes notifications@github.com wrote:

Rename to something more explicit such as ThreadLocal Disallow region name declarations of the same name Disallow RPLs containing ThreadLocal AND other RPL elements Disallow aliasing ThreadLocal with other Regions Hi Alex,

I'm not sure what the last one above means. How do two regions get aliased?

--Vikram Professor, Computer Science University of Illinois at Urbana-Champaign http://llvm.org/~vadve

atzannes commented 11 years ago

Sorry that was indeed unclear! I was thinking that pointers/references to ThreadLocal should not be passed across tasks perhaps and that it might be better to have Root:* be disjoint from ThreadLocal.