Perhaps the value used in the decoder could be defined as "FullSystem ? 1 : 0".
Steve On Mon, Feb 13, 2012 at 10:09 PM, Koan-Sin Tan <[email protected]>wrote: > it seems changing the type of FullSystem from bool to integer is easier. > > On Mon, Feb 13, 2012 at 7:03 PM, Andreas Hansson > <[email protected]> wrote: > > Hi everyone, > > > > The way the FullSystem flag is handled in the ISA decoder generation is > causing problems with llvm-gcc and clang since a boolean variable is being > used in a switch statement. Is there any good way of making this an if/else > rather than a switch? It affects all ISAs. > > > > build/ALPHA/arch/alpha/decoder.cc:7649:11: error: switch condition has > boolean > > value [-Werror] > > switch (FullSystem) { > > ^ ~~~~~~~~~~ > > build/ALPHA/arch/alpha/decoder.cc:7665:11: error: switch condition has > boolean > > value [-Werror] > > switch (FullSystem) { > > ^ ~~~~~~~~~~ > > build/ALPHA/arch/alpha/decoder.cc:7688:7: error: switch condition has > boolean > > value [-Werror] > > switch (FullSystem) { > > ^ ~~~~~~~~~~ > > > > Thanks. > > > > Andreas > > > > -- IMPORTANT NOTICE: The contents of this email and any attachments are > confidential and may also be privileged. If you are not the intended > recipient, please notify the sender immediately and do not disclose the > contents to any other person, use it for any purpose, or store or copy the > information in any medium. Thank you. > > _______________________________________________ > > gem5-dev mailing list > > [email protected] > > http://m5sim.org/mailman/listinfo/gem5-dev > > > > -- > // koan-sin tan > _______________________________________________ > gem5-dev mailing list > [email protected] > http://m5sim.org/mailman/listinfo/gem5-dev > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
