diff options
author | Björn Gustavsson <[email protected]> | 2017-08-10 14:16:20 +0200 |
---|---|---|
committer | GitHub <[email protected]> | 2017-08-10 14:16:20 +0200 |
commit | 49918dd747d6d949d72ca51f1323b04abd2afba5 (patch) | |
tree | d8db2ec58b96e98ef7001ba5aaf6ed4f40bb1340 /lib/compiler/src | |
parent | 8689569bb3964151d3734c634ea930dae9171655 (diff) | |
parent | f59e70ce61913356a67d58e03d6b68c29e983363 (diff) | |
download | otp-49918dd747d6d949d72ca51f1323b04abd2afba5.tar.gz otp-49918dd747d6d949d72ca51f1323b04abd2afba5.tar.bz2 otp-49918dd747d6d949d72ca51f1323b04abd2afba5.zip |
Merge pull request #1526 from bjorng/bjorn/compiler/non-variable-apply
sys_core_fold: Fix unsafe optimization of non-variable apply
OTP-14526
Diffstat (limited to 'lib/compiler/src')
-rw-r--r-- | lib/compiler/src/sys_core_fold.erl | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/compiler/src/sys_core_fold.erl b/lib/compiler/src/sys_core_fold.erl index e0cd6da06f..d73060fb7e 100644 --- a/lib/compiler/src/sys_core_fold.erl +++ b/lib/compiler/src/sys_core_fold.erl @@ -395,10 +395,10 @@ expr(#c_receive{clauses=Cs0,timeout=T0,action=A0}=Recv, Ctxt, Sub) -> expr(#c_apply{anno=Anno,op=Op0,args=As0}=App, _, Sub) -> Op1 = expr(Op0, value, Sub), As1 = expr_list(As0, value, Sub), - case Op1 of - #c_var{} -> + case cerl:is_data(Op1) of + false -> App#c_apply{op=Op1,args=As1}; - _ -> + true -> add_warning(App, invalid_call), Err = #c_call{anno=Anno, module=#c_literal{val=erlang}, |