-
Notifications
You must be signed in to change notification settings - Fork 1
Merge Kotlin/Native patches #7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
llvm/cmake/modules/AddLLVM.cmake
Outdated
else() | ||
set(LLVM_LINK_OR_COPY copy) | ||
endif() | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fragment seems the same as the one above. Probably the merge side effect. The commit is already present in the branch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed: I've deleted the commit
33d7de1
to
ce0dc40
Compare
The JVM uses signals as an efficient way to do GC safe-pointing. When clang is loaded as a native library on the JVM, the LLVM signal handler ends up getting triggered by the signals generated by the GC mechanism. This problem is addressed by libjsig, but that approach requires preloading the library on JVM startup, which cannot be ensured when using Gradle daemons. Furthermore, libjsig does not work on newer versions of MacOS.
* Added an option to configure MaxDevirtIterations * fixup! Added an option to configure MaxDevirtIterations (cherry picked from commit 74a4177)
The reverted commit causes a PCH handling bug on Linux (KT-75560). This reverts commit 8c74a9a.
ce0dc40
to
a4264e6
Compare
Scope: OSIP-555