[PATCH] D131413: [BOLT] Remove old layout from function layout

Fabian Parzefall via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 8 10:13:29 PDT 2022


FPar created this revision.
FPar added reviewers: maksfb, yota9, Amir, rafauler.
Herald added a subscriber: ayermolo.
Herald added a project: All.
FPar requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

To track whether a function's new layout is different from its old
layout when updating it, the old layout would be kept around in memory
indefinitely (if the new layout is different). This was used only for
debugging/logging purposes. This patch forces the caller of function
layout's update method to copy the old layout into a temporary if they
need it by removing the old layout fields.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D131413

Files:
  bolt/include/bolt/Core/FunctionLayout.h
  bolt/include/bolt/Passes/BinaryPasses.h
  bolt/lib/Core/FunctionLayout.cpp
  bolt/lib/Passes/BinaryPasses.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D131413.450859.patch
Type: text/x-patch
Size: 7349 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220808/744d3d8d/attachment.bin>


More information about the llvm-commits mailing list