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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to