[PATCH] D70287: [LLD][COMMON] Fix Incorrect Stream in LLD ErrorHandler

Erik McClure via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 14 20:27:48 PST 2019


blackhole12 added a comment.

Yes, I am calling `lld::elf::link` from my program, which statically links LLD as a library. I can create a change that adds a `diag` stream to all the linker calls, but this would touch quite a lot of files. It may be more appropriate to close this change and to create a new change for adding a new stream.

Unfortunately, I can't submit any additional patches. I have another patch for LLD that fixes a cleanup error, but https://reviews.llvm.org/differential/diff/create/ returns `Unhandled Exception ("Exception")` no matter how many times I log out, clear the cache, or use another browser. I tried using the command-line arcanist tool, but that doesn't seem to like my patch either. I've tried asking on IRC, but nobody seems able to help and I have no idea who to contact about this issue.

I can either replace this patch with one that adds an additional diagnostics stream, or I can wait until I figure out a way to submit new patches here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70287





More information about the llvm-commits mailing list