[all-commits] [llvm/llvm-project] 98f0e4: Revert "[SelectionDAG] Add pcsections recursively ...

Marco Elver via All-commits all-commits at lists.llvm.org
Fri Feb 3 10:01:23 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 98f0e4f611b40c902cb0df3ef080ae2c00e862d4
      https://github.com/llvm/llvm-project/commit/98f0e4f611b40c902cb0df3ef080ae2c00e862d4
  Author: Marco Elver <elver at google.com>
  Date:   2023-02-03 (Fri, 03 Feb 2023)

  Changed paths:
    M llvm/include/llvm/CodeGen/SelectionDAG.h
    M llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
    M llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
    R llvm/test/CodeGen/AArch64/pcsections-memtransfer.ll
    M llvm/test/CodeGen/AArch64/pcsections.ll
    R llvm/test/CodeGen/X86/pcsections-memtransfer.ll
    M llvm/test/CodeGen/X86/pcsections.ll

  Log Message:
  -----------
  Revert "[SelectionDAG] Add pcsections recursively on SDNode values"
Revert "[SelectionDAG] Add missing setValue calls in visitIntrinsicCall"

This reverts commit 0c64e1b68f36640ffe82fc90e6279c50617ad1cc.
This reverts commit 1142e6c7c795de7f80774325a07ed49bc95a48c9.

It spuriously added !pcsections where they shouldn't be. See added test
case in test/CodeGen/X86/pcsections.ll as an example. The reason is that
the SelectionDAG chains operations in a basic block as "operands"
pointing to preceding instructions. This resulted in setting the
metadata on _all_ instructions preceding the one that should have the
metadata.

Reverting for now because the semantics of !pcsections was completely
buggy now.




More information about the All-commits mailing list