MCAssembler: Simplify getSectionAddressSize

MaskRay · MaskRay · commit 36cadb613a2b · 2025-08-23T18:51:47.000-07:00
The tail fragment must be of type FT_Data. Reduce a computeFragmentSize
use.
diff --git a/llvm/lib/MC/MCAssembler.cpp b/llvm/lib/MC/MCAssembler.cpp
@@ -353,10 +353,9 @@ const MCSymbol *MCAssembler::getBaseSymbol(const MCSymbol &Symbol) const {
 }
 
 uint64_t MCAssembler::getSectionAddressSize(const MCSection &Sec) const {
-  assert(HasLayout);
-  // The size is the last fragment's end offset.
   const MCFragment &F = *Sec.curFragList()->Tail;
-  return getFragmentOffset(F) + computeFragmentSize(F);
+  assert(HasLayout && F.getKind() == MCFragment::FT_Data);
+  return getFragmentOffset(F) + F.getSize();
 }
 
 uint64_t MCAssembler::getSectionFileSize(const MCSection &Sec) const {

Original file line number	Diff line number	Diff line change
`@@ -353,10 +353,9 @@ const MCSymbol *MCAssembler::getBaseSymbol(const MCSymbol &Symbol) const {`
`353`	`353`	`}`
`354`	`354`
`355`	`355`	`uint64_t MCAssembler::getSectionAddressSize(const MCSection &Sec) const {`
`356`		`- assert(HasLayout);`
`357`		`- // The size is the last fragment's end offset.`
`358`	`356`	`const MCFragment &F = *Sec.curFragList()->Tail;`
`359`		`- return getFragmentOffset(F) + computeFragmentSize(F);`
	`357`	`+ assert(HasLayout && F.getKind() == MCFragment::FT_Data);`
	`358`	`+ return getFragmentOffset(F) + F.getSize();`
`360`	`359`	`}`
`361`	`360`
`362`	`361`	`uint64_t MCAssembler::getSectionFileSize(const MCSection &Sec) const {`