Reuse temporary files for print-changed=diff
authorJamie Schmeiser <schmeise@ca.ibm.com>
Thu, 27 May 2021 14:19:13 +0000 (10:19 -0400)
committerJamie Schmeiser <schmeise@ca.ibm.com>
Thu, 27 May 2021 14:19:13 +0000 (10:19 -0400)
Summary:
Make the file name and descriptors static so that they are reused by
print-changed=diff. This avoids errors about being unable to create
temporary files when doing the later comparisons in a large compile.

Author: Jamie Schmeiser <schmeise@ca.ibm.com>
Reviewed By: aeubanks (Arthur Eubanks)
Differential Revision: https://reviews.llvm.org/D100116

llvm/lib/Passes/StandardInstrumentations.cpp

index 0410fdb..b28f4ba 100644 (file)
@@ -132,8 +132,8 @@ std::string doSystemDiff(StringRef Before, StringRef After,
   // Store the 2 bodies into temporary files and call diff on them
   // to get the body of the node.
   const unsigned NumFiles = 3;
-  std::string FileName[NumFiles];
-  int FD[NumFiles]{-1, -1, -1};
+  static std::string FileName[NumFiles];
+  static int FD[NumFiles]{-1, -1, -1};
   for (unsigned I = 0; I < NumFiles; ++I) {
     if (FD[I] == -1) {
       SmallVector<char, 200> SV;