[PATCH] D32689: DEF: migrate def parser from LLD to LLVM

Dean Michael Berris via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 30 18:32:37 PDT 2017


dberris added inline comments.


================
Comment at: include/llvm/Object/COFFImportFile.h:84
+
+  bool operator==(const COFFShortExport &E) {
+    return (Name == E.Name && ExtName == E.ExtName &&
----------------
This probably should be a non-member friend function, and have an `operator !=(...)` overload as well.


Repository:
  rL LLVM

https://reviews.llvm.org/D32689





More information about the llvm-commits mailing list