[all-commits] [llvm/llvm-project] 4397b7: Revert "Re-reapply "[DebugInfo] Use variadic debug...

Nathan Chancellor via All-commits all-commits at lists.llvm.org
Fri Apr 30 20:43:30 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 4397b7095d640f9b9426c4d0135e999c5a1de1c5
      https://github.com/llvm/llvm-project/commit/4397b7095d640f9b9426c4d0135e999c5a1de1c5
  Author: Nathan Chancellor <nathan at kernel.org>
  Date:   2021-04-30 (Fri, 30 Apr 2021)

  Changed paths:
    M llvm/include/llvm/IR/DebugInfoMetadata.h
    M llvm/include/llvm/IR/Instructions.h
    M llvm/include/llvm/IR/IntrinsicInst.h
    M llvm/include/llvm/IR/Operator.h
    M llvm/include/llvm/Transforms/Utils/Local.h
    M llvm/lib/CodeGen/AsmPrinter/DebugLocEntry.h
    M llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
    M llvm/lib/IR/DebugInfoMetadata.cpp
    M llvm/lib/IR/Instructions.cpp
    M llvm/lib/IR/IntrinsicInst.cpp
    M llvm/lib/IR/Operator.cpp
    M llvm/lib/Transforms/Coroutines/CoroFrame.cpp
    M llvm/lib/Transforms/Utils/Local.cpp
    M llvm/test/DebugInfo/NVPTX/debug-info.ll
    R llvm/test/DebugInfo/salvage-gep.ll
    R llvm/test/DebugInfo/salvage-nonconst-binop.ll
    M llvm/test/Transforms/InstCombine/debuginfo-sink.ll
    M llvm/test/Transforms/Reassociate/undef_intrinsics_when_deleting_instructions.ll

  Log Message:
  -----------
  Revert "Re-reapply "[DebugInfo] Use variadic debug values to salvage BinOps and GEP instrs with non-const operands""

This reverts commit 791930d74087b8ae8901172861a0fd21a211e436, as per
https://llvm.org/docs/DeveloperPolicy.html#patch-reversion-policy.

I observed breakage with the Linux kernel, as reported at
https://reviews.llvm.org/D91722#2724321

Fixes exist at
https://reviews.llvm.org/D101523
https://reviews.llvm.org/D101540

but they have not landed so to unbreak the tree for the weekend, revert
this commit.

Commit b11e4c990771 ("Revert "[DebugInfo] Drop DBG_VALUE_LISTs with an
excessive number of debug operands"") only reverted one follow-up fix,
not the original patch that broke the kernel.

e




More information about the All-commits mailing list