SuccessChanges

Changes from Git (git http://labmaster3.local/git/llvm-project.git)

Summary

  1. GlobalModuleIndex - Fix use-after-move clang static analyzer warning. (details)
  2. [X86AsmBackend] Be consistent about placing definitions out of line (details)
  3. [X86AsmBackend] Move static function before sole use [NFC] (details)
  4. [X86] Adjust nop emission by compiler to consider target decode (details)
Commit fce887beb79780d0e0b19e8ab6176978a3dce9b8 by llvm-dev
GlobalModuleIndex - Fix use-after-move clang static analyzer warning.
Shadow variable names meant we were referencing the Buffer input
argument, not the GlobalModuleIndex member that its std::move()'d it.
The file was modifiedclang/lib/Serialization/GlobalModuleIndex.cpp
Commit 6cb3957730e9085bb7c37d871c790f910efdd6a7 by listmail
[X86AsmBackend] Be consistent about placing definitions out of line
[NFC]
The file was modifiedllvm/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp
Commit 563d3e344452c8923db09b043b8db471fc413b1e by listmail
[X86AsmBackend] Move static function before sole use [NFC]
The file was modifiedllvm/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp
Commit 1d641daf260308815d014d1bf1b424a1ed1e7277 by listmail
[X86] Adjust nop emission by compiler to consider target decode
limitations
The primary motivation of this change is to bring the code more closely
in sync behavior wise with the assembler's version of nop emission.  I'd
like to eventually factor them into one, but that's hard to do when one
has features the other doesn't.
The longest encodeable nop on x86 is 15 bytes, but many processors - for
instance all intel chips - can't decode the 15 byte form efficiently.
On those processors, it's better to use either a 10 byte or 11 byte
sequence depending.
The file was modifiedllvm/test/MC/X86/stackmap-nops.ll
The file was modifiedllvm/test/CodeGen/X86/stackmap-nops.ll
The file was modifiedllvm/lib/Target/X86/X86MCInstLower.cpp
The file was modifiedllvm/test/CodeGen/X86/align-branch-boundary-suppressions.ll