----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3662/ -----------------------------------------------------------
(Updated Oct. 11, 2016, 9:06 p.m.) Review request for Default. Summary (updated) ----------------- syscall_emul: move SyscallDesc into its own .hh and .cc files Repository: gem5 Description (updated) ------- Changeset 11674:30214967f119 --------------------------- syscall_emul: move SyscallDesc into its own .hh and .cc files The class was crammed into syscall_emul.hh which has tons of forward declarations and template definitions. To clean it up a bit, moved the class into separate files and commented the class with doxygen style comments. Also, provided some encapsulation by adding some accessors and a mutator. The syscallreturn.hh file was renamed syscall_return.hh to make it consistent with other similarly named files in the src/sim directory. The DPRINTF_SYSCALL macro was moved into its own header file with the include the Base and Verbose flags as well. Diffs (updated) ----- src/sim/syscall_desc.cc PRE-CREATION src/sim/syscall_emul.hh 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/sim/syscall_emul.cc 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/sim/syscallreturn.hh 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/arch/alpha/linux/process.cc 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/arch/arm/freebsd/process.cc 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/arch/arm/linux/process.cc 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/arch/mips/linux/process.cc 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/arch/power/linux/process.cc 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/arch/sparc/linux/process.cc 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/arch/sparc/linux/syscalls.cc 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/arch/sparc/solaris/process.cc 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/arch/x86/linux/process.cc 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/arch/x86/process.cc 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/kern/tru64/tru64.hh 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/sim/SConscript 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/sim/process.hh 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/sim/process.cc 220fa4099b9a91526b8a1828f27cf1a9f3c15837 src/sim/syscall_debug_macros.hh PRE-CREATION src/sim/syscall_desc.hh PRE-CREATION Diff: http://reviews.gem5.org/r/3662/diff/ Testing ------- Thanks, Brandon Potter _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
