[all-commits] [llvm/llvm-project] cf7eac: [ObjectSizeOffsetVisitor] Bail after visiting 100 ...

Arthur Eubanks via All-commits all-commits at lists.llvm.org
Wed Sep 27 05:54:55 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: cf7eac9650f3ce40154e5e35734a147599495af4
      https://github.com/llvm/llvm-project/commit/cf7eac9650f3ce40154e5e35734a147599495af4
  Author: Arthur Eubanks <aeubanks at google.com>
  Date:   2023-09-27 (Wed, 27 Sep 2023)

  Changed paths:
    M llvm/include/llvm/Analysis/MemoryBuiltins.h
    M llvm/lib/Analysis/MemoryBuiltins.cpp
    A llvm/test/Transforms/DeadStoreElimination/object-size-offset-visitor-max-visit-instructions.ll

  Log Message:
  -----------
  [ObjectSizeOffsetVisitor] Bail after visiting 100 instructions (#67479)

We're running into stack overflows for huge functions with lots of phis.
Even without the stack overflows, this is recursing >7000 in some
auto-generated code.

This fixes the stack overflow and brings down the compile time to
something reasonable.




More information about the All-commits mailing list