[all-commits] [llvm/llvm-project] 26ba1f: [DebugInfo][test] Change two MIR tests to use -sta...

[Fangrui Song via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 26ba1f77b55e7a961acc05d94bfa4b677a9e5d83
      https://github.com/llvm/llvm-project/commit/26ba1f77b55e7a961acc05d94bfa4b677a9e5d83
  Author: Fangrui Song <maskray at google.com>
  Date:   2020-01-19 (Sun, 19 Jan 2020)

  Changed paths:
    M llvm/test/DebugInfo/ARM/cfi-eof-prologue.mir
    M llvm/test/DebugInfo/X86/debug-loc-asan.mir

  Log Message:
  -----------
  [DebugInfo][test] Change two MIR tests to use -start-before=livedebugvalues instead of -start-after=patchable-function

To break order dependency between livedebugvalues and patchable-function.


  Commit: a72d15e37c5e066f597f13a8ba60aff214ac992d
      https://github.com/llvm/llvm-project/commit/a72d15e37c5e066f597f13a8ba60aff214ac992d
  Author: Fangrui Song <maskray at google.com>
  Date:   2020-01-19 (Sun, 19 Jan 2020)

  Changed paths:
    M llvm/include/llvm/Target/Target.td

  Log Message:
  -----------
  [XRay] Set hasSideEffects flag of PATCHABLE_FUNCTION_{ENTER,EXIT}

Otherwise they may be picked as the delay slot by mips-delay-slot-filler, if we move patchable-function before mips-delay-slot-filler.


  Commit: 9a24488cb67a90f889529987275c5e411ce01dda
      https://github.com/llvm/llvm-project/commit/9a24488cb67a90f889529987275c5e411ce01dda
  Author: Fangrui Song <maskray at google.com>
  Date:   2020-01-19 (Sun, 19 Jan 2020)

  Changed paths:
    M llvm/lib/CodeGen/TargetPassConfig.cpp
    M llvm/test/CodeGen/AArch64/O0-pipeline.ll
    M llvm/test/CodeGen/AArch64/O3-pipeline.ll
    A llvm/test/CodeGen/AArch64/patchable-function-entry-bti.ll
    M llvm/test/CodeGen/ARM/O3-pipeline.ll
    M llvm/test/CodeGen/X86/O0-pipeline.ll
    M llvm/test/CodeGen/X86/O3-pipeline.ll

  Log Message:
  -----------
  [CodeGen] Move fentry-insert, xray-instrumentation and patchable-function before addPreEmitPass()

This intention is to move patchable-function before aarch64-branch-targets
(configured in AArch64PassConfig::addPreEmitPass) so that we emit BTI before NOPs
(see https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92424).

This also allows addPreEmitPass() passes to know the precise instruction sizes if they want.

Tried x86-64 Debug/Release builds of ccls with -fxray-instrument -fxray-instruction-threshold=1.
No output difference with this commit and the previous commit.


Compare: https://github.com/llvm/llvm-project/compare/5fa2022ec005...9a24488cb67a