Don't print uselistorder in --print-changed

[aeubanks]

Summary

Using uselistorders is fairly niche, it shouldn't be on by default and mostly just clutters the output.

[jamieschmeiser]

As I recall, I added this in based on comments in the review and I am fine with removing it. However, the parameter to unWrapAndPrint was added to preserve existing function and if this is removed, then the parameter should also be removed as it will always be false. The parameter can be dropped from the call to print also, as it defaults to false.

[aeubanks]

address comments

[ychen]

I was suggesting this to catch non-determinism issues like the one D64866 fixed. Sounds good to me to make it off by default. Any way to make it opt-in? Making it part of API (add a field to PrintPassOptions) would be the best but it could be a follow-up patch with a FIXME added to this patch.

[aeubanks]

In D111282#3048567, @ychen wrote:

I was suggesting this to catch non-determinism issues like the one D64866 fixed. Sounds good to me to make it off by default. Any way to make it opt-in? Making it part of API (add a field to PrintPassOptions) would be the best but it could be a follow-up patch with a FIXME added to this patch.

Yeah, adding it as a separate flag sounds good if anybody needs it

[jamieschmeiser]

LGTM, thanks

llvm/lib/Passes/StandardInstrumentations.cpp

 template <typename IRUnitT>
 void TextChangeReporter<IRUnitT>::handleInitialIR(Any IR) {
   // Always print the module.
   // Unwrap and print directly to avoid filtering problems in general routines.
   auto *M = unwrapModule(IR, /*Force=*/true);
   assert(M && "Expected module to be unwrapped when forced.");
   Out << "*** IR Dump At Start ***\n";
   M->print(Out, nullptr,
-           /*ShouldPreserveUseListOrder=*/true);
+           /*ShouldPreserveUseListOrder=*/false);
 }
 
 template <typename IRUnitT>
 void TextChangeReporter<IRUnitT>::omitAfter(StringRef PassID,
                                             std::string &Name) {
   Out << formatv("*** IR Dump After {0} on {1} omitted because no change ***\n",
                  PassID, Name);
 }
if (PrintChanged == ChangePrinter::PrintChangedVerbose ||
       PrintChanged == ChangePrinter::PrintChangedQuiet)
     TextChangeReporter<std::string>::registerRequiredCallbacks(PIC);
 }
 
 void IRChangedPrinter::generateIRRepresentation(Any IR, StringRef PassID,
                                                 std::string &Output) {
   raw_string_ostream OS(Output);
   unwrapAndPrint(OS, IR,
-                 /*ShouldPreserveUseListOrder=*/true);
+                 /*ShouldPreserveUseListOrder=*/false);
   OS.str();
 }
 
 void IRChangedPrinter::handleAfter(StringRef PassID, std::string &Name,
                                    const std::string &Before,
                                    const std::string &After, Any) {
   // Report the IR before the changes when requested.
   if (PrintChangedBefore)