[all-commits] [llvm/llvm-project] cefbc2: [Inliner] Remove alloca merging

aeubanks via All-commits all-commits at lists.llvm.org
Thu Mar 16 10:03:36 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: cefbc2e0e794847a0634d3ad7ce94a8e72d0569d
      https://github.com/llvm/llvm-project/commit/cefbc2e0e794847a0634d3ad7ce94a8e72d0569d
  Author: Arthur Eubanks <aeubanks at google.com>
  Date:   2023-03-16 (Thu, 16 Mar 2023)

  Changed paths:
    M llvm/lib/Transforms/IPO/Inliner.cpp

  Log Message:
  -----------
  [Inliner] Remove alloca merging

See https://discourse.llvm.org/t/rfc-we-should-stop-merging-allocas-in-the-inliner/41975 for the RFC from 2016 around this.

Backend stack coloring takes care of what this initially set out to help with.

This only affects the legacy inliner, which for the most part doesn't matter anymore. The new PM inliner never looked at this flag.

There are some backends running the legacy always-inline pass, but that's unlikely to matter.

Reviewed By: asbirlea

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




More information about the All-commits mailing list