Project

General

Profile

Actions

Bug #17836

closed

JIT fails to create a precompiled header with Clang

Added by xtkoba (Tee KOBAYASHI) about 1 year ago. Updated 5 months ago.

Status:
Closed
Priority:
Normal
Target version:
-
ruby -v:
ruby 3.1.0dev (2021-04-24T02:17:20Z master dee58d7ae7) [x86_64-linux]
[ruby-core:103629]

Description

A log message:

Creating precompiled header
Starting process: /usr/lib/llvm/12/bin/clang /usr/lib/llvm/12/bin/clang --target=x86_64-pc-linux-gnu -w -funwind-tables -fdeclspec -Wfatal-errors -fPIC -shared -w -pipe -Oz -fno-fast-math -emit-pch -o /tmp/_ruby_mjit_hp16974u0.h.gch /var/tmp/ruby/xxx/include/ruby-3.1.0/x86_64-linux/rb_mjit_min_header-3.1.0.h
clang-12: fatal error: cannot specify -o when generating multiple output files

According to https://lists.boost.org/boost-users/2017/01/87080.php, the -c option is (sometimes) needed to emit PCH with Clang. A patch is attached for that.


Files

ruby-mjit-clang.patch (387 Bytes) ruby-mjit-clang.patch xtkoba (Tee KOBAYASHI), 04/27/2021 11:30 PM

Updated by k0kubun (Takashi Kokubun) about 1 year ago

  • Assignee set to k0kubun (Takashi Kokubun)
Actions #2

Updated by k0kubun (Takashi Kokubun) about 1 year ago

  • Status changed from Open to Closed

Applied in changeset git|9f8a50723f8a84b3e4755b418570148f422d1b28.


Specify -c to emit pch with clang (#4423)

[Bug #17836]

Updated by k0kubun (Takashi Kokubun) about 1 year ago

Sorry, I forgot adding Co-authored-by in the commit message (please consider using GitHub if you care), but I merged your patch. Thanks.

Actions #4

Updated by nagachika (Tomoyuki Chikanaga) 6 months ago

  • Backport changed from 2.6: UNKNOWN, 2.7: UNKNOWN, 3.0: UNKNOWN to 2.7: UNKNOWN, 3.0: REQUIRED
Actions #5

Updated by nagachika (Tomoyuki Chikanaga) 6 months ago

  • Backport changed from 2.7: UNKNOWN, 3.0: REQUIRED to 2.7: REQUIRED, 3.0: REQUIRED

Updated by nagachika (Tomoyuki Chikanaga) 6 months ago

  • Backport changed from 2.7: REQUIRED, 3.0: REQUIRED to 2.7: REQUIRED, 3.0: DONE

ruby_3_0 69272c74a3f675373fc6c6f32b685620da8a85ee merged revision(s) 9f8a50723f8a84b3e4755b418570148f422d1b28.

Updated by usa (Usaku NAKAMURA) 5 months ago

  • Backport changed from 2.7: REQUIRED, 3.0: DONE to 2.7: DONE, 3.0: DONE

ruby_2_7 57f8bbcf63f8e2ef5c2cea488ba21c3e7c34f862 merged revision(s) 9f8a50723f8a84b3e4755b418570148f422d1b28.

Actions

Also available in: Atom PDF