Woohoo, this is the last :)

Happy to see more ErrorOr/std::error_code becoming Error/Expected 😊

In D64687#1584337, @sylvestre.ledru wrote:

I went ahead and committed your revert in r365992. Thanks!

Next time, please remove the clang/ from your patch
arc could not apply it:

lld-link: error: could not open 'libcmt.lib': No such file or directory

Do you find another use case of printDynamicString? If yes, can you name it?

I committed this for you in rL365981 (it may have broken a Windows build for a long time. I wanted to fix it soon..). Thanks!

In rL365872#670363, @JamesNagurne wrote:

In rL365872#670346, @RKSimon wrote:

@MaskRay This is causing EXPENSIVE_CHECKS failures - please can you investigate ?

http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-win/builds/18594/steps/test-check-all/logs/stdio

Looks like the default implementation of getSupportedSanitizers explicitly masks out SanitizerKind::Vptr, which would cause this error for any systems which have a default target that do not support the sanitizer.
If it's just this specific sanitizer, the fix should be to trivially revert the change presented here.

Sorry, I accidentally updated this one by mistake. vptr is not always available. Thanks!

Using ccache and distcc together is a common pattern.

Update tests

D64282 + D64283 are the last 2 patches to make ieeelongdouble work.

This should be rebased after D64294.

Use A->render(Args, CmdArgs);

I did this in rC365862 with a test after my refactoring of the interface rC365860.

Fix title

Update title and description

Add doc to InitLLVM.h
assert set_new_handler returns nullptr

Fix output of empty sequence/map

I think there are trailing spaces in these YAML files (e.g. in Object/Inputs/COFF/section-aux-symbol.yaml, header: is followed by 10 spaces). When you commit the patch, it might to good to delete them :) D64566