Issue 174258
Summary EquivalenceClasses: Sample code incorrect?
Labels new issue
Assignees
Reporter shevek-google
    https://github.com/llvm/llvm-project/blob/main/llvm/include/llvm/ADT/EquivalenceClasses.h#L50

Does this need to be `(*it)->isLeader()` and `.member_begin(**it)`? All the `*`s are missing.

See, for example:
* https://github.com/tensorflow/tensorflow/blob/master/tensorflow/compiler/mlir/tensorflow/transforms/convert_control_to_data_outputs.cc#L428
* https://github.com/tensorflow/tensorflow/blob/master/tensorflow/compiler/mlir/tensorflow/transforms/convert_control_to_data_outputs.cc#L449

I get the impression this contract changed at some point, but I didn't dig into it to find out when.
_______________________________________________
llvm-bugs mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs

Reply via email to