Matthew Poremba has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/44825 )

Change subject: configs: apu_se.py hotfix
......................................................................

configs: apu_se.py hotfix

Missed two optparse -> argparse changes. Square runs.

Change-Id: I3a652380e4c4202a376413602fa3698a28ff9206
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/44825
Maintainer: Matthew Poremba <[email protected]>
Reviewed-by: Daniel Carvalho <[email protected]>
Tested-by: kokoro <[email protected]>
---
M configs/example/apu_se.py
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Daniel Carvalho: Looks good to me, approved
  Matthew Poremba: Looks good to me, approved
  kokoro: Regressions pass



diff --git a/configs/example/apu_se.py b/configs/example/apu_se.py
index b0da8df..f779df3 100644
--- a/configs/example/apu_se.py
+++ b/configs/example/apu_se.py
@@ -181,14 +181,14 @@
 #--     0   1   x   UC_L2   (Uncached_GL2)
 #--     0   0   x   UC_All  (Uncached_All_Load)
 # default value: 5/C_RO_S (only allow caching in GL2 for read. Shared)
-parser.add_argument("--m-type", type='int', default=5,
+parser.add_argument("--m-type", type=int, default=5,
help="Default Mtype for GPU memory accesses. This is the " "value used for all memory accesses on an APU and is the " "default mode for dGPU unless explicitly overwritten by "
                     "the driver on a per-page basis.  Valid values are "
                     "between 0-7")

-parser.add_argument("--gfx-version", type="string", default='gfx801',
+parser.add_argument("--gfx-version", type=str, default='gfx801',
                     help="Gfx version for gpu: gfx801, gfx803, gfx900")

 Ruby.define_options(parser)

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/44825
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I3a652380e4c4202a376413602fa3698a28ff9206
Gerrit-Change-Number: 44825
Gerrit-PatchSet: 2
Gerrit-Owner: Matthew Poremba <[email protected]>
Gerrit-Reviewer: Daniel Carvalho <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: Matthew Poremba <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to