formatting fixes

Doc1 => DocLHS; Doc2 => DocRHS; made raw_ostream argument in compareFiles and printDifferences; lowercase argument prompts;

fix error handling in llvm-tapi-diff.cpp

removed import that was auto-added by text editor causing failure of tests.

fixed heading on DiffEngine.cpp; made helper to collect single scalar values; ReturnedOutput => Output; made implementation details private for class variables; made helper to set order indicator for print functions; moved functions that did not need to be in header; removed unneeded conditional; removed unneeded scope; DiffVec => Values, TargValues, DocValues; addValToDiffVec => addDiffToTargetSlice; removed unnecessary this-> calls; removed typo in comment; fixed heading on llvm-tapi-diff.cpp; fixed error handling in llvm-tapi-diff.cpp

fixed brackets for regex in test

use regex to try to fix windows failure.

update separators in tests

uploading patch again to trigger rebuild

rebased patch from main to fix patch apply issue, added new line for CMakeLists.txt

added new lines to test input files.

llvm.MachO.PlatformKind => llvm::MachO::PlatformKind in platform.cpp; removed 1 and 2 in favor of LHS and RHS in DiffEngine.cpp; refactored comments in DiffEngine.h; changed DiffOutput from class to struct; added llvm_unreachable in switch statements in DiffEngine.h; Added new lines at end of files.

clang-format fixes and replaced macho.dylib for macho.yaml

fixed patch upload for -U999999

updated header descriptions, updated error handling for convertFileToBinary and if a file cannot be cast as TapiUniversal.

fixed issue with getOSAndEnvironmentName

fixed issue with applying patch

resubmitted patch because of failure

changed getTargetTriple to getTargetTripleName; fixed header descriptions; fixed unreachable path; return const reference from getInterfaceFile; passing Target as const reference into getTargetTripleName; added llvm_unreachable to getOSAndEnvironmentName; NOT -> not in tests; removed unnecessary comparison in checkSymbolEquality; refactored typenames for addValToDiffVec template function; added Doxygen comments for DiffEngine.h; One and Two => LHS and RHS in llvm-tapi-diff; added WithColor; return llvm::Error or std::unique_ptr<Binary> for convertFileToBinary instead of only dumping error; used EXIT_FAILURE instead of returning 1;

uploaded with -U999999

fix clang-tidy warnings

fixed issue with ... at end of tbd files that caused malformed file for testing

clang-format fixes and included getOSAndEnvironmentName.

moved transform wrapper to TextStubHelpers, refactored inequality tests for both v3 and v4.

refactored tests to implement wrapper, removed single line {}, changed Document.insert to Document.emplace

updated std::lower_bound to llvm::lower_bound