[llvm] [MC] Move MCFragment::Atom to MCSection::Atoms (PR #95341)
Fangrui Song via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 13 10:05:10 PDT 2024
================
@@ -111,6 +113,9 @@ class MCSection {
// subsections.
SmallVector<std::pair<unsigned, FragList>, 1> Subsections;
+ // Mach-O only: the defining non-temporary symbol for each fragment.
+ SmallVector<const MCSymbol *, 0> Atoms;
----------------
MaskRay wrote:
We could, but MCSection.cpp doesn't include MCSectionMachO.h and therefore cannot use `Atoms`.
We could make MCSection.cpp include MCSectionMachO.h, but there is still going to be Mach-O concept in generic code.
On the other hand, the number of MCSection instances is small.
https://github.com/llvm/llvm-project/pull/95341
More information about the llvm-commits
mailing list