[clang] [llvm] Trying to fix undefined symbol error caused by iterator variable (PR #141507)

via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 4 06:46:38 PDT 2025


ShashwathiNavada wrote:

> This looks basically right. But I'm not sure all the conditions for calling it are correct. Is getDeclareMapperVarRef relevant? Does it matter what scope we're inserting the variable into? (Can the variable be inserted in a namespace? Can it be a local variable?)

@efriedma-quic  I've added the condition to check for local variables as you suggested. However, I believe getDeclareMapperVarRef() is relevant in this context, since I'm handling iterators declared in #pragma omp declare mapper. Let me know if you think any further changes are needed.

https://github.com/llvm/llvm-project/pull/141507


More information about the llvm-commits mailing list