[PATCH] D87272: [lld] Buffer writes when composing a single diagnostic

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 8 09:00:35 PDT 2020


MaskRay added inline comments.


================
Comment at: lld/Common/ErrorHandler.cpp:216
+    if (errorLimit == 0 || errorCount <= errorLimit) {
+      raw_ostream &errs = lld::errs();
+      SmallString<256> buf;
----------------
jhenderson wrote:
> This code is very similar to the warning code. Can you de-duplicate it with a helper function?
`os` is used after the common part and requires a `SmallString`, so it is a bit difficult to share code.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87272/new/

https://reviews.llvm.org/D87272



More information about the llvm-commits mailing list