[PATCH] D129242: [clang-repl] Add host exception support check utility flag.

2022-07-29 Thread Mikael Holmén via Phabricator via cfe-commits
uabelho added a comment. In D129242#3687968 , @sunho wrote: > https://reviews.llvm.org/D130788 this is the patch fyi. Great, the patch works for us. Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D12

[PATCH] D129242: [clang-repl] Add host exception support check utility flag.

2022-07-29 Thread Sunho Kim via Phabricator via cfe-commits
sunho added a comment. https://reviews.llvm.org/D130788 this is the patch fyi. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129242/new/ https://reviews.llvm.org/D129242 ___ cfe-commits mailing list cfe-

[PATCH] D129242: [clang-repl] Add host exception support check utility flag.

2022-07-29 Thread Sunho Kim via Phabricator via cfe-commits
sunho added a comment. @uabelho The thing is we have to statically link libstdc++ in JIT stack, but this is not possible at the moment in many platforms. It's actually possible with a new generation linker called JITLink, but it has limited platform support. For now, I just sumbitted a patch di

[PATCH] D129242: [clang-repl] Add host exception support check utility flag.

2022-07-29 Thread Sunho Kim via Phabricator via cfe-commits
sunho added a comment. In D129242#3686713 , @uabelho wrote: > In D129242#3686610 , @sunho wrote: > >> @uabelho Hi, that's a typeinfo symbol of libc++ abi. It's quite weird that >> __cxa_throw is availble while _Z

[PATCH] D129242: [clang-repl] Add host exception support check utility flag.

2022-07-29 Thread Mikael Holmén via Phabricator via cfe-commits
uabelho added a comment. In D129242#3686610 , @sunho wrote: > @uabelho Hi, that's a typeinfo symbol of libc++ abi. It's quite weird that > __cxa_throw is availble while _ZTIPKc is not. Do you have some special > setting regarding libc++ in your environm

[PATCH] D129242: [clang-repl] Add host exception support check utility flag.

2022-07-29 Thread Sunho Kim via Phabricator via cfe-commits
sunho added a comment. @uabelho Hi, that's a typeinfo symbol of libc++ abi. It's quite weird that __cxa_throw is availble while _ZTIPKc is not. Do you have some special setting regarding libc++ in your environment? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://revi

[PATCH] D129242: [clang-repl] Add host exception support check utility flag.

2022-07-28 Thread Mikael Holmén via Phabricator via cfe-commits
uabelho added a comment. Hello, The new testcase added in this patch fails for me all the time. I'm compiling on a linux RHEL 7 machine. When I run the clang-repl command in the test I get JIT session error: Symbols not found: [ _ZTIPKc ] error: Failed to materialize symbols: { (main, { _Z14

[PATCH] D129242: [clang-repl] Add host exception support check utility flag.

2022-07-28 Thread Sunho Kim via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG3cc3be8fa471: [clang-repl] Add host exception support check utility flag. (authored by sunho). Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo