[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