[PATCH] D45801: COFF: Use (name, output characteristics) as a key when grouping input sections into output sections.

[Shoaib Meenai via Phabricator via llvm-commits]

smeenai added inline comments.


================
Comment at: lld/COFF/Writer.cpp:458
+    // DATA | R | W.
+    if (Name == ".CRT")
+      OutChars = DATA | R;
----------------
If I'm reading this correctly link.exe only has this special case in 32-bit mode, but we're doing it for all architectures?

Also, when you say 32-bit, do you mean x86 specifically, or 32-bit ARM as well?


https://reviews.llvm.org/D45801