sim: always generate sim/tags.cc

Information
Submitter:	Curtis Dunham
Repository:	gem5
Branch:	default
Bugs:
Depends On:
Reviewers
Groups:	Default
People:

Description

scons: always generate sim/tags.cc

Due to insufficient build deps, the checkpoint tags might not get
updated; this patch solves this. Due to the uncommon nature of the
build target, regenerating tags.cc is a fairly clean solution. Since
SCons hashes file contents, it won't recompile anything unless a new
checkpoint upgrader is actually added.

Testing Done

Change Summary:

"checkpointer" -> "checkpoint upgrader"

Description:

~		Due to insufficient build deps, the checkpoint tags might not get updated; this patch solves this. Due to the uncommon nature of the build target, regenerating tags.cc is a fairly clean solution. Since SCons hashes file contents, it won't recompile anything unless a new checkpointer is actually added.
	~	Due to insufficient build deps, the checkpoint tags might not get updated; this patch solves this. Due to the uncommon nature of the build target, regenerating tags.cc is a fairly clean solution. Since SCons hashes file contents, it won't recompile anything unless a new checkpoint upgrader is actually added.

Seems reasonable.

Could you please line wrap the patch description, though?

Change Summary:

Massaged description text

Given current Ship-It, planning to commit at end of week.

Description:

~		Due to insufficient build deps, the checkpoint tags might not get updated; this patch solves this. Due to the uncommon nature of the build target, regenerating tags.cc is a fairly clean solution. Since SCons hashes file contents, it won't recompile anything unless a new checkpoint upgrader is actually added.
	~	scons: always generate sim/tags.cc
	+
	+	Due to insufficient build deps, the checkpoint tags might not get
	+	updated; this patch solves this. Due to the uncommon nature of the
	+	build target, regenerating tags.cc is a fairly clean solution. Since
	+	SCons hashes file contents, it won't recompile anything unless a new
	+	checkpoint upgrader is actually added.

You have a pending review.

Review Board 2.0.15

This change has been marked as submitted.

Screenshots

Files

Change Summary:

Description:

Change Summary:

Description:

Status: Closed (submitted)