Skip to content

Commit

Permalink
Merge pull request #2392 from fenollp/nohipe
Browse files Browse the repository at this point in the history
drop deprecated HiPE compilation

(cherry picked from commit b7cbd44)
  • Loading branch information
michaelklishin committed Jun 25, 2020
1 parent 3ec6db5 commit b24452b
Show file tree
Hide file tree
Showing 7 changed files with 3 additions and 253 deletions.
18 changes: 0 additions & 18 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -64,24 +64,6 @@ define PROJECT_ENV
{exit_on_close, false}
]},
{halt_on_upgrade_failure, true},
{hipe_compile, false},
%% see bug 24513 [in legacy Bugzilla] for how this list was created
{hipe_modules,
[rabbit_reader, rabbit_channel, gen_server2, rabbit_exchange,
rabbit_command_assembler, rabbit_framing_amqp_0_9_1, rabbit_basic,
rabbit_event, lists, queue, priority_queue, rabbit_router,
rabbit_trace, rabbit_misc, rabbit_binary_parser,
rabbit_exchange_type_direct, rabbit_guid, rabbit_net,
rabbit_amqqueue_process, rabbit_variable_queue,
rabbit_binary_generator, rabbit_writer, delegate, gb_sets, lqueue,
sets, orddict, rabbit_amqqueue, rabbit_limiter, gb_trees,
rabbit_queue_index, rabbit_exchange_decorator, gen, dict, ordsets,
file_handle_cache, rabbit_msg_store, array,
rabbit_msg_store_ets_index, rabbit_msg_file,
rabbit_exchange_type_fanout, rabbit_exchange_type_topic, mnesia,
mnesia_lib, rpc, mnesia_tm, qlc, sofs, proplists, credit_flow,
pmon, ssl_connection, tls_connection, ssl_record, tls_record,
gen_fsm, ssl]},
{ssl_apps, [asn1, crypto, public_key, ssl]},
%% see rabbitmq-server#114
{mirroring_flow_control, true},
Expand Down
6 changes: 0 additions & 6 deletions docs/rabbitmq.conf.example
Original file line number Diff line number Diff line change
Expand Up @@ -466,12 +466,6 @@
## NB: Change these only if you understand what you are doing!
##

## Explicitly enable/disable hipe compilation. This option is deprecated
## because HiPE supported has been
## effectively discontinued in Erlang 22.
##
# hipe_compile = false

## Timeout used when waiting for Mnesia tables in a cluster to
## become available.
##
Expand Down
23 changes: 0 additions & 23 deletions docs/rabbitmqctl.8
Original file line number Diff line number Diff line change
Expand Up @@ -2084,29 +2084,6 @@ to the connected client:
.Pp
Evaluates an Erlang expression on the target node
.\" ------------------------------------------------------------------
.It Cm hipe_compile Ar directory
.Pp
Performs HiPE-compilation and caches resulting
.Pa .beam Ns -files in the given directory.
.Pp
Parent directories are created if necessary.
Any existing
.Pa .beam
files from the directory are automatically deleted prior to compilation.
.Pp
To use this precompiled files, you should set
.Ev RABBITMQ_SERVER_CODE_PATH
environment variable to directory specified in
.Cm hipe_compile
invocation.
.Pp
For example, to HiPE-compile modules and store them to
.Pa /tmp/rabbit-hipe/ebin
directory:
.sp
.Dl rabbitmqctl hipe_compile /tmp/rabbit-hipe/ebin
.El
.\" ------------------------------------------------------------------
.\" ## Queue Operations
.\" ------------------------------------------------------------------
.Ss Queue Operations
Expand Down
2 changes: 2 additions & 0 deletions priv/schema/rabbit.schema
Original file line number Diff line number Diff line change
Expand Up @@ -1070,6 +1070,8 @@ end}.
%% Explicitly enable/disable hipe compilation.
%%
%% {hipe_compile, true},
%%
%% DEPRECATED: this is a no-op and is kept only to allow old configs.

{mapping, "hipe_compile", "rabbit.hipe_compile",
[{datatype, {enum, [true, false]}}]}.
Expand Down
10 changes: 1 addition & 9 deletions src/rabbit.erl
Original file line number Diff line number Diff line change
Expand Up @@ -353,15 +353,7 @@ run_prelaunch_second_phase() ->
%% 3. Logging.
ok = rabbit_prelaunch_logging:setup(Context),

case IsInitialPass of
true ->
%% 4. HiPE compilation.
ok = rabbit_prelaunch_hipe:setup(Context);
false ->
ok
end,

%% 5. Clustering.
%% 4. Clustering.
ok = rabbit_prelaunch_cluster:setup(Context),

%% Start Mnesia now that everything is ready.
Expand Down
171 changes: 0 additions & 171 deletions src/rabbit_hipe.erl

This file was deleted.

26 changes: 0 additions & 26 deletions src/rabbit_prelaunch_hipe.erl

This file was deleted.

0 comments on commit b24452b

Please sign in to comment.