[lld] [lld][macho] Rename test to fix spelling (PR #160938)

Vincent Lee via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 26 16:29:25 PDT 2025


================
@@ -1687,6 +1687,9 @@ void CStringSection::writeTo(uint8_t *buf) const {
 
 void CStringSection::finalizeContents() {
   uint64_t offset = 0;
+  // TODO: Call buildCStringPriorities() to support cstring ordering when
----------------
thevinster wrote:

Okay that's fair. We'd have to read in the orderfile at some point when specified. In theory, we could thread that information of whether an orderfile contains cstring up to here, but it might not be worth that complexity if this specific case is something a user would even care about. 

https://github.com/llvm/llvm-project/pull/160938


More information about the llvm-commits mailing list