[PATCH] D40407: [COFF] Implement constructor priorities

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 27 14:54:45 PST 2017


mstorsjo added inline comments.


================
Comment at: lib/CodeGen/TargetLoweringObjectFileImpl.cpp:535-536
     if (Priority != 65535) {
-      Name += '.';
-      Name += utostr(65535 - Priority);
+      raw_string_ostream OS(Name);
+      OS << format(".%05u", 65535 - Priority);
     }
----------------
ruiu wrote:
> nit: I believe you could write this as
> 
>       raw_string_ostream(Name) << format(".%05u", 65535 - Priority);
Oh, yes - thanks!


https://reviews.llvm.org/D40407





More information about the llvm-commits mailing list