[PATCH] D37577: [libclang] add 'clang_getCursorTLSKind'

[Masud Rahman via Phabricator via cfe-commits]

frutiger added inline comments.


================
Comment at: bindings/python/tests/cindex/test_tls_kind.py:14
+int tls_none;
+thread_local tls_dynamic;
+""", lang = 'cpp')
----------------
frutiger wrote:
> compnerd wrote:
> > Can we add a test case for static TLS as well please?  Also, I think that we should add a test case for `__declspec(thread)`.
> I will add a test case for static TLS.
> 
> `__declspec(thread)` is not implemented as a TLS kind on a cursor; instead it is implemented as a child attribute of the cursor.  Would you like me to add a test case for that here?
Please ignore the above comment regarding `__declspec(thread)`, it does indeed add a static TLS kind to the cursor.


https://reviews.llvm.org/D37577