[all-commits] [llvm/llvm-project] 040c1b: Move EntryExitInstrumentation pass location

aeubanks via All-commits all-commits at lists.llvm.org
Mon Mar 1 10:08:40 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 040c1b49d7a7eaa23d883ca363744944f5597d92
      https://github.com/llvm/llvm-project/commit/040c1b49d7a7eaa23d883ca363744944f5597d92
  Author: Arthur Eubanks <aeubanks at google.com>
  Date:   2021-03-01 (Mon, 01 Mar 2021)

  Changed paths:
    M clang/lib/CodeGen/BackendUtil.cpp
    M clang/test/CodeGen/X86/x86_64-instrument-functions.c
    M clang/test/CodeGen/mcount.c
    M clang/test/Frontend/gnu-mcount.c
    M llvm/include/llvm/Transforms/Utils/EntryExitInstrumenter.h
    M llvm/lib/CodeGen/TargetPassConfig.cpp
    M llvm/lib/Transforms/IPO/PassManagerBuilder.cpp
    M llvm/test/CodeGen/AArch64/O0-pipeline.ll
    M llvm/test/CodeGen/AArch64/O3-pipeline.ll
    M llvm/test/CodeGen/AMDGPU/opt-pipeline.ll
    M llvm/test/CodeGen/ARM/O3-pipeline.ll
    M llvm/test/CodeGen/ARM/gnu_mcount_nc.ll
    M llvm/test/CodeGen/Mips/long-call-mcount.ll
    M llvm/test/CodeGen/Mips/mcount.ll
    M llvm/test/CodeGen/PowerPC/mcount-insertion.ll
    M llvm/test/CodeGen/X86/O0-pipeline.ll
    M llvm/test/CodeGen/X86/musttail-inalloca.ll
    M llvm/test/CodeGen/X86/opt-pipeline.ll
    M llvm/test/Other/opt-O0-pipeline-enable-matrix.ll
    M llvm/test/Other/opt-O0-pipeline.ll
    M llvm/test/Other/opt-O2-pipeline.ll
    M llvm/test/Other/opt-O3-pipeline-enable-matrix.ll
    M llvm/test/Other/opt-O3-pipeline.ll
    M llvm/test/Other/opt-Os-pipeline.ll

  Log Message:
  -----------
  Move EntryExitInstrumentation pass location

This seems to be more of a Clang thing rather than a generic LLVM thing,
so this moves it out of LLVM pipelines and as Clang extension hooks into
LLVM pipelines.

Move the post-inline EEInstrumentation out of the backend pipeline and
into a late pass, similar to other sanitizer passes. It doesn't fit
into the codegen pipeline.

Also fix up EntryExitInstrumentation not running at -O0 under the new
PM. PR49143

Reviewed By: hans

Differential Revision: https://reviews.llvm.org/D97608




More information about the All-commits mailing list