[all-commits] [llvm/llvm-project] 67f9c3: Make two texts static in `ReplayInlineAdvisor` (#7...

Tacet via All-commits all-commits at lists.llvm.org
Thu Jan 25 17:04:45 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 67f9c35ffee1aea3fa8a18b57a7b27400a03435a
      https://github.com/llvm/llvm-project/commit/67f9c35ffee1aea3fa8a18b57a7b27400a03435a
  Author: Tacet <advenam.tacet at trailofbits.com>
  Date:   2024-01-26 (Fri, 26 Jan 2024)

  Changed paths:
    M llvm/lib/Analysis/ReplayInlineAdvisor.cpp

  Log Message:
  -----------
  Make two texts static in `ReplayInlineAdvisor` (#79489)

This commit makes two variables static.
That makes two buildbot tests pass with short string annotations.
I suspect that there may be use after end of life bug and it's fixed by
this change, but it requires confirmation.

Short string annotations PR (reverted):
- https://github.com/llvm/llvm-project/pull/79049

Tests fixed with this PR:
```
  LLVM :: Transforms/Inline/cgscc-inline-replay.ll 
  LLVM :: Transforms/SampleProfile/inline-replay.ll
```
Buildbot output:
https://lab.llvm.org/buildbot/#/builders/5/builds/40364/steps/9/logs/stdio

This PR does not resolve a problem with `Clang :: SemaCXX/builtins.cpp`,
related PR is:
- https://github.com/llvm/llvm-project/pull/79522




More information about the All-commits mailing list