[PATCH] D46672: COFF: ICF a section and its associated pdata section as a unit.

[Shoaib Meenai via Phabricator via llvm-commits]

smeenai added inline comments.


================
Comment at: lld/COFF/ICF.cpp:141
+  };
+  return ChildClasses(A) == ChildClasses(B);
+}
----------------
Is there a possibility of ChildClasses(A) and ChildClasses(B) having the same contents but in a different order? Would we want to ICF in that case? (Since right now the == is going to be doing an ordered comparison.)


https://reviews.llvm.org/D46672