[llvm-commits] Please review: adding Function attribute "ThreadSafety"

[Chris Lattner]

On Feb 4, 2013, at 6:00 AM, Kostya Serebryany <kcc at google.com> wrote:

> [resurrecting an old thread... again]
> 
> Chris, Bill, 
> 
> Is the new Attribute implementation good enough to support two more function attributes? 
> We need two attributes do enable/disable tsan and msan with function granularity. 
> Just like with asan, where we have AddressSafety attribute mapped to __attribute__((no_address_safety_analysis)), 
> I propose to add ThreadSafety attribute (will map to the existing __attribute__((no_thread_safety_analysis)))
> and UninitializedChecks attribute (will map to a new __attribute__((no_uninitialized_checks)))
> (naming is hard, I'd love to hear better suggestions). 

It's fine with me in principle, the right place to bikeshed on the name is the cfe-dev list.

-Chris