SuccessChanges

Summary

  1. Hopefully last fix for bot failures (details)
  2. [llvm][MIRVRegNamerUtils] Adding hashing on FrameIndex MachineOperands. (details)
  3. [X86][Disassembler] Fix a bug when disassembling an empty string (details)
  4. Add a couple of missed wildcards in debug-pass-manager output checking (details)
  5. Revert "[ThinLTO] Add additional ThinLTO pipeline testing with new PM" (details)
Commit 7aed43b60739653b13b8503f9df4c958c44feed8 by tejohnson
Hopefully last fix for bot failures
Hopefully final bot fix for last few failures from
2af97be8027a0823b88d4b6a07fc5eedb440bc1f.
Looks like sometimes the "llvm::" preceeding objects get printed in the
debug pass manager output and sometimes they don't. Replace with
wildcard matching.
The file was modifiedllvm/test/Other/new-pm-thinlto-prelink-samplepgo-defaults.ll (diff)
The file was modifiedclang/test/CodeGen/thinlto-distributed-newpm.ll (diff)
The file was modifiedllvm/test/Other/new-pm-thinlto-postlink-pgo-defaults.ll (diff)
The file was modifiedllvm/test/Other/new-pm-thinlto-postlink-samplepgo-defaults.ll (diff)
The file was modifiedllvm/test/Other/new-pm-thinlto-prelink-pgo-defaults.ll (diff)
Commit 484a7472f1aa6906f2b66dc33bcf69cc8d5b9f29 by puyan
[llvm][MIRVRegNamerUtils] Adding hashing on FrameIndex MachineOperands.
This patch makes it so that cases where multiple instructions that
differ only in their FrameIndex MachineOperand values no longer collide.
For instance:
%1:_(p0) = G_FRAME_INDEX %stack.0
%2:_(p0) = G_FRAME_INDEX %stack.1
Prior to this patch these instructions would collide together.
Differential Revision: https://reviews.llvm.org/D71583
The file was modifiedllvm/lib/CodeGen/MIRVRegNamerUtils.cpp (diff)
The file was addedllvm/test/CodeGen/MIR/X86/mir-namer-hash-frameindex.mir
Commit 64a93afc3c630c39e5c583e4f67aef5821d635b6 by maskray
[X86][Disassembler] Fix a bug when disassembling an empty string
readPrefixes() assumes insn->bytes is non-empty. The code path is not
exercised in llvm-mc because llvm-mc does not feed empty input to
MCDisassembler::getInstruction().
This bug is uncovered by a5994c789a2982a770254ae1607b5b4cb641f73c. An
empty string did not crash before because the deleted regionReader()
allowed UINT64_C(-1) as insn->readerCursor.
  Bytes.size() <= Address -> R->Base
0 <= UINT64_C(-1) - UINT32_C(-1)
The file was modifiedllvm/lib/Target/X86/Disassembler/X86Disassembler.cpp (diff)
The file was modifiedllvm/unittests/MC/Disassembler.cpp (diff)
Commit cb988a858abbaf1a1ae0fe03f2a1dae692131ea9 by tejohnson
Add a couple of missed wildcards in debug-pass-manager output checking
Along with the previous fix for bot failures from
2af97be8027a0823b88d4b6a07fc5eedb440bc1f, need to add a wildcard in a
couple of places where my local output did not print "llvm::" but the
bot is.
The file was modifiedclang/test/CodeGen/thinlto-distributed-newpm.ll (diff)
The file was modifiedllvm/test/Other/new-pm-thinlto-prelink-pgo-defaults.ll (diff)
Commit 6288f86e870c7bb7fe47cc138320b9eb34c93941 by tejohnson
Revert "[ThinLTO] Add additional ThinLTO pipeline testing with new PM"
This reverts commit 2af97be8027a0823b88d4b6a07fc5eedb440bc1f.
After attempting to fix bot failures from matching issues (mostly due to
inconsistent printing of "llvm::" prefixes on objects, and
AnalysisManager objects being printed differntly, I am now seeing some
differences I don't understand (real differences in the passes being
printed). Giving up at this point to allow the bots to recover. Will
revisit later.
The file was removedllvm/test/Other/Inputs/new-pm-thinlto-prelink-pgo-defaults.proftext
The file was removedllvm/test/Other/Inputs/new-pm-thinlto-samplepgo-defaults.prof
The file was removedllvm/test/Other/new-pm-thinlto-postlink-samplepgo-defaults.ll
The file was modifiedllvm/test/Other/new-pm-pgo.ll (diff)
The file was removedllvm/test/Other/new-pm-thinlto-postlink-pgo-defaults.ll
The file was removedclang/test/CodeGen/thinlto-distributed-newpm.ll
The file was removedllvm/test/Other/new-pm-thinlto-prelink-samplepgo-defaults.ll
The file was removedllvm/test/Other/new-pm-thinlto-prelink-pgo-defaults.ll