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
[JSC][32bit] Fix JIT being disabled when compiling JSC on linux with clang #10089
[JSC][32bit] Fix JIT being disabled when compiling JSC on linux with clang #10089
Conversation
EWS run on previous version of this PR (hash 2fc2d41) |
2fc2d41
to
c958b32
Compare
EWS run on previous version of this PR (hash c958b32) |
c958b32
to
a437cff
Compare
EWS run on previous version of this PR (hash a437cff) |
a437cff
to
09d1351
Compare
EWS run on previous version of this PR (hash 09d1351) |
09d1351
to
ad34acc
Compare
EWS run on current version of this PR (hash ad34acc) |
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.
r=me
ad34acc
to
20ba989
Compare
β¦clang https://bugs.webkit.org/show_bug.cgi?id=234034 Reviewed by Justin Michaud. JIT was always disabled when building JSC with clang on linux 32 bits because CMake checked if the compiler defined __thumb2__ and while gcc does define it by default, clang doesn't. This patch adds extra compilation flags when compiling JSC with clang for arm (except on Darwin), so the expected __thumb2__ define is defined and JIT is enabled. Also some minor changes: (1) this patch also adds __ARM_ARCH_8__ and __ARM_ARCH_8A__ to the list that sets BTHUMB_ARCH_VERSION and WTF_THUMB_ARCH_VERSION to 4. These two seem to be used mostly for sanity check. (2) a small fix in AirArg where clang complains about a missing typename. gcc seems to accept either. * Source/JavaScriptCore/b3/air/AirArg.h: (JSC::B3::Air::Arg::isValidAddrForm): * Source/JavaScriptCore/wasm/WasmAirIRGeneratorBase.h: (JSC::Wasm::parseAndCompileAirImpl): * Source/WTF/wtf/PlatformCPU.h: * Source/bmalloc/bmalloc/BPlatform.h: * Source/cmake/OptionsCommon.cmake: Canonical link: https://commits.webkit.org/260921@main
20ba989
to
7dd18bd
Compare
Committed 260921@main (7dd18bd): https://commits.webkit.org/260921@main Reviewed commits have been landed. Closing PR #10089 and removing active labels. |
7dd18bd
ad34acc