I have only minor remarks but I'd like you to check if my hunch is correct and the proposed modifications will fix fix PR49777 *and* fix PR49779.
Also, the number of arguments need to be increased, let's go big and automatic here.
This broke the windows LLDB bot. Please fix it soon or revert.
Rename TargetInfo.allowDebugInfoForExternalVar to TargetInfo.allowDebugInfoForExternalRef.
I'm not really sure about the test, my local setup didn't have CUDA attached properly but this should work in principle ;)
Also, it seems that the newly added test "coro-debug-dbg.values-O2-nouse.ll" passes even without this patch
I only have small nits, nothing major. @fghanim, is this OK with you, if so, please accept?
If it works, that'll be good enough for me :)
Could you update the description of this patch as well?
It's still not fully clear to me what problem this patch is aiming at resolving.
remove trailing whitespace
remove trailing whitesepace
LGTM, assuming @craig.topper is also happy.
use larger address space for iossim