usx95 wrote:

> Ugh, I wonder if this is unintentional and we should change the 
> implementation of these data structures. I also find this behavior very 
> surprising. 

I can try changing the data structure. Let me do that in a separate PR. It 
would need an extra Value comparison which needs to be done [while inserting 
the 
element](https://github.com/llvm/llvm-project/blob/e2467cbfa8fdc440debac1cfb930ee476f3d5e42/llvm/include/llvm/ADT/ImmutableSet.h#L542)
 and this could be a slight concern for other users. Let me give it a try.

https://github.com/llvm/llvm-project/pull/159582
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to