[clang] [llvm] [Clang][OpenMP] This is addition fix for #92210. (PR #94802)

Alexey Bataev via cfe-commits cfe-commits at lists.llvm.org
Sun Jul 7 12:22:50 PDT 2024


alexey-bataev wrote:

> Hi Alexey,
> 
> Thank you take look!
> 
> 
> 
> > Missed one thing, the assignments for boolean flags must be |=, otherwise we may miss some combinations.
> 
> 
> 
> Are you talking about assignment of following?
> 
> HasMapBasePtr = any_of(M, [](c..
> 
> 
> 
> Do I need to change to:
> 
> HasMapBasePtr |= any_of(M, [](c
> 
> 
> 
> Thanks.  
> 
> Jennifer

Yes, I thought it matches the original code behavior.

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


More information about the cfe-commits mailing list