[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