----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/986/ -----------------------------------------------------------
(Updated Jan. 29, 2012, 10:18 p.m.) Review request for Default. Changes ------- 1. Andreas Hansson provided informative description for the patch 2. remove -Wno-mismatched-tags and make class/struct consistent 3. some small changes discussed during review Description (updated) ------- clang: Enable compiling gem5 using clang 2.9 and 3.0 This patch adds the necessary flags to the SConstruct and SConscript files for compiling using clang 2.9 and later (on Ubuntu et al and OSX XCode 4.2), and also cleans up a bunch of compiler warnings found by clang. Most of the warnings are related to hidden virtual functions, comparisons with unsigneds >= 0, and if-statements with empty bodies. A number of mismatches between struct and class are also fixed. clang 2.8 is not working as it has problems with class names that occur in multiple namespaces (e.g. Statistics in kernel_stats.hh). clang has a bug (http://llvm.org/bugs/show_bug.cgi?id=7247) which causes confusion between the container std::set and the function Packet::set, and this is currently addressed by not including the entire namespace std, but rather selecting e.g. "using std::vector" in the appropriate places. Diffs (updated) ----- SConstruct 0d5d77cfd7071145714ed24fe40890fdb2a5693b ext/libelf/SConscript 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/SConscript 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/arch/alpha/tlb.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/arch/alpha/tlb.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/arch/arm/insts/static_inst.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/arch/arm/insts/vfp.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/arch/arm/isa/templates/basic.isa 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/arch/arm/miscregs.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/arch/generic/memhelpers.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/arch/mips/faults.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/arch/mips/faults.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/arch/x86/bios/acpi.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/arch/x86/bios/intelmp.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/arch/x86/bios/intelmp.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/arch/x86/bios/smbios.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/base/fast_alloc.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/base/range_map.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/base/remote_gdb.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/base/stl_helpers.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/base.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/base.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/func_unit.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/inorder/cpu.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/inorder/cpu.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/inorder/resource.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/inorder/resource.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/inorder/resource_pool.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/inorder/resource_pool.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/inorder/resources/cache_unit.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/inorder/thread_context.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/nativetrace.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/bpred_unit.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/commit.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/cpu.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/cpu.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/decode.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/decode_impl.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/fetch.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/fu_pool.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/fu_pool.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/iew.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/iew_impl.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/inst_queue.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/inst_queue_impl.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/lsq.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/lsq_unit.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/mem_dep_unit.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/mem_dep_unit.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/rename.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/o3/sat_counter.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/quiesce_event.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/sched_list.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/simple/atomic.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/simple/atomic.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/simple/base.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/simple/base.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/simple/timing.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/simple/timing.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/cpu/static_inst.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/dev/alpha/tsunami_cchip.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/dev/alpha/tsunami_io.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/dev/arm/pl111.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/dev/arm/pl111.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/dev/copy_engine.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/dev/disk_image.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/dev/disk_image.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/dev/ide_ctrl.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/dev/ns_gige.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/dev/pciconfigall.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/dev/pcidev.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/mem/cache/base.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/mem/cache/base.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/mem/cache/tags/iic.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/mem/cache/tags/iic_repl/gen.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/mem/cache/tags/iic_repl/gen.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/mem/cache/tags/iic_repl/repl.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/mem/packet.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/mem/ruby/system/Sequencer.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/python/m5/SimObject.py 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/sim/core.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/sim/process.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/sim/process.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/sim/process_impl.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/sim/serialize.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/sim/sim_object.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/sim/sim_object.cc 0d5d77cfd7071145714ed24fe40890fdb2a5693b src/sim/syscall_emul.hh 0d5d77cfd7071145714ed24fe40890fdb2a5693b Diff: http://reviews.gem5.org/r/986/diff/diff Testing ------- Thanks, Koan-Sin Tan _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
