[PATCH] D47467: [IR] Begin removal of TerminatorInst by removing successor manipulation.

Dean Michael Berris via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 29 04:48:02 PDT 2018


dberris added inline comments.


================
Comment at: llvm/include/llvm/IR/CFG.h:193
+
+  inline bool operator==(const Self &x) const { return Idx == x.Idx; }
+
----------------
Maybe this should be a non-member friend, and works with const and non-const versions of compatible type?

```
friend inline bool operator==(const Self& L, const Self& R) {
  return L.Idx == R.Idx;
}
```


Repository:
  rL LLVM

https://reviews.llvm.org/D47467





More information about the llvm-commits mailing list