[m-rev.] for review: multi-cons-id switch arms for MLDS

Peter Wang novalazy at gmail.com
Wed Aug 26 14:17:26 AEST 2009


On 2009-08-26, Julien Fischer <juliensf at csse.unimelb.edu.au> wrote:
> 
> On Wed, 19 Aug 2009, Zoltan Somogyi wrote:
> 
> >Implement multi-arm switches for the MLDS backend.

Committed this.

Branches: main

compiler/handle_options.m:
        Allow multi-arm switches on Java backend.

diff --git a/compiler/handle_options.m b/compiler/handle_options.m
index 1890c41..04797b4 100644
--- a/compiler/handle_options.m
+++ b/compiler/handle_options.m
@@ -1133,16 +1133,17 @@ postprocess_options_2(OptionTable0, Target, GC_Method, TagsMethod0,
         option_implies(highlevel_code, mutable_always_boxed, bool(no),
             !Globals),
 
-        % Currently, multi-arm switches % have been tested only for the LLDS
+        % Currently, multi-arm switches have been tested only for the LLDS
         % backend (which always generates C) and for the MLDS backend when
-        % it is generating C code.
+        % it is generating C or Java code.
         (
-            Target = target_c
+            ( Target = target_c
+            ; Target = target_java
+            )
         ;
             ( Target = target_x86_64
             ; Target = target_asm
             ; Target = target_il
-            ; Target = target_java
             ; Target = target_erlang
             ),
             globals.set_option(allow_multi_arm_switches, bool(no), !Globals)

--------------------------------------------------------------------------
mercury-reviews mailing list
Post messages to:       mercury-reviews at csse.unimelb.edu.au
Administrative Queries: owner-mercury-reviews at csse.unimelb.edu.au
Subscriptions:          mercury-reviews-request at csse.unimelb.edu.au
--------------------------------------------------------------------------



More information about the reviews mailing list