Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/6041
Change subject: tests: Resurrect initest input file(s).
..
tests: Resurrect initest input file(s).
Delete the current version of
Alec Roelke has uploaded this change for review. (
https://gem5-review.googlesource.com/6026
Change subject: arch-riscv: Move compressed ops out of ISA
..
arch-riscv: Move compressed ops out of ISA
This patch moves static
Alec Roelke has uploaded this change for review. (
https://gem5-review.googlesource.com/6021
Change subject: arch-riscv: Move static_inst into a directory
..
arch-riscv: Move static_inst into a directory
This patch creates
Alec Roelke has uploaded this change for review. (
https://gem5-review.googlesource.com/6023
Change subject: arch-riscv: Move unknown out of ISA description
..
arch-riscv: Move unknown out of ISA description
This patch
Alec Roelke has uploaded this change for review. (
https://gem5-review.googlesource.com/6025
Change subject: arch-riscv: Remove static parts of AMOs out of ISA
..
arch-riscv: Remove static parts of AMOs out of ISA
This
Alec Roelke has uploaded this change for review. (
https://gem5-review.googlesource.com/6022
Change subject: arch-riscv: Move standard ops out of ISA
..
arch-riscv: Move standard ops out of ISA
This patch removes static
Alec Roelke has uploaded this change for review. (
https://gem5-review.googlesource.com/6024
Change subject: arch-riscv: Move parts of mem insts out of ISA
..
arch-riscv: Move parts of mem insts out of ISA
This patch moves
Alec Roelke has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5322 )
Change subject: arch-riscv: Add missing system calls
..
arch-riscv: Add missing system calls
This patch adds all system
Alec Roelke has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5501 )
Change subject: sim-se: Add default to SyscallDesc constructor
..
sim-se: Add default to SyscallDesc constructor
This
I thought about this a bit more, and one thing that would be worse under
this scheme would be that generated source files would be generated
multiple times when building multiple builds since they'd no longer be in a
common location. I think it's pretty rare for people to build multiple
builds,
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/6001
Change subject: tests: Fix the stats unit test.
..
tests: Fix the stats unit test.
This has been broken since February. The
Gabe Black has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5483 )
Change subject: sparc: Move integer StaticInst base classes out of the ISA
desc.
..
sparc: Move integer StaticInst base
Gabe Black has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5482 )
Change subject: sparc: Move the mem base classes out of the ISA description.
..
sparc: Move the mem base classes out of the
Gabe Black has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5481 )
Change subject: sparc: Move the microop/macroop base classes out of the ISA
desc.
..
sparc: Move the microop/macroop
Gabe Black has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5465 )
Change subject: sparc: Return debug faults from unimplemented instructions.
..
sparc: Return debug faults from
Gabe Black has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5464 )
Change subject: sparc: Pull the unimplemented formats out of the ISA
description.
..
sparc: Pull the unimplemented
Gabe Black has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5463 )
Change subject: sparc: Pull the "Uknown" StaticInst class out of the ISA
description.
..
sparc: Pull the "Uknown"
Gabe Black has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5462 )
Change subject: sparc: Pull most of the Nop format out of the ISA
description.
..
sparc: Pull most of the Nop format out
Gabe Black has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5461 )
Change subject: sparc: Pull more StaticInst base classes out of the ISA
desc.
..
sparc: Pull more StaticInst base
Gabe Black has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5422 )
Change subject: sparc: Pull flat static instruction classes out of the ISA.
..
sparc: Pull flat static instruction classes
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/5981
Change subject: scons: Move some compiler flag setting code to the
SConstruct.
..
scons: Move some compiler flag setting code to
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/5985
Change subject: scons: Minor cleanup of how partial linking is handled in
makeEnv.
..
scons: Minor cleanup of how partial linking
Hello Jason Lowe-Power, Andreas Sandberg,
I'd like you to reexamine a change. Please visit
https://gem5-review.googlesource.com/5822
to look at the new patch set (#3).
Change subject: scons: Switch from "guards" to "tags" on source files.
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/5983
Change subject: scons: Remove the extra_deps option from the helper
function make_obj.
..
scons: Remove the extra_deps option
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/5984
Change subject: scons: Break make_obj into make_static and make_shared
functions.
..
scons: Break make_obj into make_static and
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/5982
Change subject: scons: Get rid of a flag which makes Werror optional.
..
scons: Get rid of a flag which makes Werror optional.
This
Ok, sounds good. I was thinking that sort of thing would make sense, but
didn't want to go that way unilaterally.
Gabe
On Tue, Nov 21, 2017 at 6:05 AM, Andreas Sandberg
wrote:
> Hi Gabe,
>
> I won't have time to review them anytime soon. Since only Jason has
> offered
Andreas Sandberg has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5764 )
Change subject: arch-arm: ArmPMU refactor
..
arch-arm: ArmPMU refactor
Change the definition of PMU events in order
Andreas Sandberg has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5762 )
Change subject: cpu, cpu, sim: move Cycle probe update
..
cpu, cpu, sim: move Cycle probe update
Move the code
Andreas Sandberg has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5763 )
Change subject: arch-arm: Do not increment PMU cycle event in WFI/WFE
..
arch-arm: Do not increment PMU cycle event
Austin Harris has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5901 )
Change subject: sim: Fix need to save address space info during
serialization.
..
sim: Fix need to save address space
Hey Gabe,
I don't have any strong feelings on this, just a couple of points.
1) I think we should support "backwards compatibility" if we do this by
having symlinks from the old names to the new names. This way we don't
break all of the scripts people have been using.
2) How much effort is this
Giacomo Travaglini has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5941 )
Change subject: scons: Build error introduced by site_tools/default.py
..
scons: Build error introduced by
Giacomo Travaglini has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5861 )
Change subject: arch-arm: Fix MSR/MRS disassemble
..
arch-arm: Fix MSR/MRS disassemble
This patch is fixing the
Giacomo Travaglini has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5862 )
Change subject: arch-arm: Fix MCR/MRC disassemble
..
arch-arm: Fix MCR/MRC disassemble
This patch is fixing the
Hi Gabe,
I won't have time to review them anytime soon. Since only Jason has
offered to review some of them, I'd suggest that we go for a more
relaxed review policy for SPARC. I would be happy for you to self
approve these changes since you are the SPARC maintainer and there are
no other obvious
Hi Gabe,
I can definitely see the reasoning behind this. I think revisiting this
is a good idea. However, I'm a bit reluctant since it affects a lot of
existing build scripts and test infrastructure.
Another benefit that you didn't mention is that this avoid some annoying
"issues" when the
Nikos Nikoleris has submitted this change and it was merged. (
https://gem5-review.googlesource.com/5881 )
Change subject: cpu-o3: Prevent cpu from suspending if it is already
draining
..
cpu-o3: Prevent cpu from
Giacomo Travaglini has uploaded this change for review. (
https://gem5-review.googlesource.com/5941
Change subject: scons: Build error introduced by site_tools/default.py
..
scons: Build error introduced by
Hello Andreas Sandberg,
I'd like you to do a code review. Please visit
https://gem5-review.googlesource.com/5921
to review the following change.
Change subject: arch-arm: HVC instruction undefined in secure EL1
..
* build/RISCV/tests/opt/quick/se/00.hello/riscv/linux/simple-atomic: FAILED!
*
build/RISCV/tests/opt/quick/se/02.insttest/riscv/linux-rv64c/simple-timing:
FAILED!
* build/RISCV/tests/opt/quick/se/00.hello/riscv/linux/simple-timing-ruby:
FAILED!
*
41 matches
Mail list logo