[all-commits] [llvm/llvm-project] 5e3ac7: Loop names used in reporting can grow very large

Jamie Schmeiser via All-commits all-commits at lists.llvm.org
Fri Sep 9 10:45:34 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5e3ac7969039678fc017a5599b051ea2b78075a4
      https://github.com/llvm/llvm-project/commit/5e3ac7969039678fc017a5599b051ea2b78075a4
  Author: Jamie Schmeiser <schmeise at ca.ibm.com>
  Date:   2022-09-09 (Fri, 09 Sep 2022)

  Changed paths:
    M clang/test/CodeGen/thinlto-distributed-newpm.ll
    M llvm/lib/Passes/StandardInstrumentations.cpp
    M llvm/test/Other/loop-pass-ordering.ll
    M llvm/test/Other/loop-pass-printer.ll
    M llvm/test/Other/loopnest-pass-ordering.ll
    M llvm/test/Other/new-pm-lto-defaults.ll
    M llvm/test/Other/new-pm-thinlto-defaults.ll
    M llvm/test/Other/pass-pipeline-parsing.ll
    M llvm/test/Other/print-before-after.ll
    M llvm/test/Transforms/LoopPredication/invalidate-analyses.ll
    M llvm/test/Transforms/LoopPredication/preserve-bpi.ll
    M llvm/test/Transforms/LoopRotate/pr35210.ll
    M llvm/test/Transforms/LoopUnroll/revisit.ll
    M llvm/test/Transforms/LoopUnroll/unroll-loop-invalidation.ll
    M llvm/test/Transforms/SimpleLoopUnswitch/nontrivial-unswitch-markloopasdeleted.ll

  Log Message:
  -----------
  Loop names used in reporting can grow very large

Summary:
The code for generating a name for loops for various reporting scenarios
created a name by serializing the loop into a string.  This may result in
a very large name for a loop containing many blocks.  Use the getName()
function on the loop instead.

Author: Jamie Schmeiser <schmeise at ca.ibm.com>
Reviewed By: Whitney (Whitney Tsang), aeubanks (Arthur Eubanks)
Differential Revision: https://reviews.llvm.org/D133587




More information about the All-commits mailing list