[PATCH] D96629: [llvm][TextAPI] add equality operator for InterfaceFile

[Cyndy Ishida via Phabricator via llvm-commits]

cishida added inline comments.


================
Comment at: llvm/unittests/TextAPI/TextStubV4Tests.cpp:1120
+
+void resetFiles(InterfaceFile &FileA, InterfaceFile &FileB,
+                const char *TBDv4File) {
----------------
cishida wrote:
> These functions are the same except variable name between these files, we have a `TextStubHelpers.h` file in the unit tests, can we just move these there? 
Can we avoid re-reading in the same input string to reset these variables and instead have one variable in the test that doesn't get altered and pass that to reassign the altered ones. 


================
Comment at: llvm/unittests/TextAPI/TextStubV4Tests.cpp:1133
+
+TargetList getTargets() {
+  return {Target(AK_i386, PlatformKind::macOS),
----------------
cishida wrote:
> If this is refactoring the multiple times we use this, it should just be a variable we can call not a function. 
sorry, I mean a const variable we can just refer to. 


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96629/new/

https://reviews.llvm.org/D96629