Fix shell test because we were taking rhs on equals

Jlalond · Jlalond · commit ae18f136f46d · 2025-11-06T13:42:38.000-08:00
diff --git a/lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp b/lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp
@@ -151,7 +151,7 @@ lldb::SectionSP MergeSections(lldb::SectionSP lhs, lldb::SectionSP rhs) {
   // We want to take the greater of two sections. If LHS and RHS are both
   // SHT_NOBITS, we should default to LHS. If RHS has a bigger section,
   // indicating it has data that wasn't stripped, we should take that instead.
-  return lhs->GetFileSize() > rhs->GetFileSize() ? lhs : rhs;
+  return rhs->GetFileSize() > lhs->GetFileSize() ? rhs : lhs;
 }
 } // end anonymous namespace
 

Original file line number	Diff line number	Diff line change
`@@ -151,7 +151,7 @@ lldb::SectionSP MergeSections(lldb::SectionSP lhs, lldb::SectionSP rhs) {`
`151`	`151`	`// We want to take the greater of two sections. If LHS and RHS are both`
`152`	`152`	`// SHT_NOBITS, we should default to LHS. If RHS has a bigger section,`
`153`	`153`	`// indicating it has data that wasn't stripped, we should take that instead.`
`154`		`- return lhs->GetFileSize() > rhs->GetFileSize() ? lhs : rhs;`
	`154`	`+ return rhs->GetFileSize() > lhs->GetFileSize() ? rhs : lhs;`
`155`	`155`	`}`
`156`	`156`	`} // end anonymous namespace`
`157`	`157`