[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