branch predictor: move out of o3 and inorder cpus

Information
Submitter:	Nilay Vaish
Repository:	gem5
Branch:	default
Bugs:
Depends On:
Reviewers
Groups:	Default
People:

Description

Changeset 9476:059046fdc03d
---------------------------
branch predictor: move out of o3 and inorder cpus
This patch moves the branch predictor files in the o3 and inorder directories
to src/cpu/pred. This allows sharing the branch predictor across different
cpu models.

This patch was originally posted by Timothy Jones in July 2010
but never made it to the repository.

Testing Done

Hi Nilay,

Thanks for resurrecting this. In principle I have no issues, I just would like to make sure that the moved code includes all the latest fixes to the branch predictor that numerous people have contributed to. If so and you get no stats differences please commit.

Thanks,
Ali

Ali Saidi Jan. 15, 2013, 2:49 a.m. (Jan. 15, 2013, 2:49 a.m.)

I'm still not sure why stats changes should occur. There may be some differences, but since a lot more effort has been put in verifying the o3 cpu branch predictor by people like Nathanael, it would be good to keep that one around and make it work with the in-order cpu or add the methods as appropriate.

Description:

~		Changeset 9383:7a322ca1288f
	~	Changeset 9476:059046fdc03d

		branch predictor: move out of o3 and inorder cpus
		This patch moves the branch predictor files in the o3 and inorder directories
		to src/cpu/pred. This allows sharing the branch predictor across different
		cpu models.

		This patch was originally posted by Timothy Jones in July 2010
		but never made it to the repository.

Diff:

Revision 2 (+1205 -1906)

Show changes

	configs/common/O3_ARM_v7a.py
	src/cpu/BaseCPU.py
	src/cpu/inorder/InOrderCPU.py
	src/cpu/inorder/SConscript
	src/cpu/inorder/inorder_cpu_builder.cc
	src/cpu/inorder/resources/bpred_unit.hh
	src/cpu/inorder/resources/bpred_unit.cc
	src/cpu/inorder/resources/branch_predictor.hh
	23 more

You have a pending review.

Review Board 2.0.15

This change has been marked as submitted.

Screenshots

Files

Description:

Changeset 9383:7a322ca1288f

Changeset 9476:059046fdc03d

Diff:

Status: Closed (submitted)