Re: [gem5-dev] Review Request 3756: cpu: Result refactoring

2017-01-26 Thread Jason Lowe-Power
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3756/#review9322 --- Ship it! Ship It! - Jason Lowe-Power On Jan. 26, 2017, 1:40 p.m.,

Re: [gem5-dev] Review Request 3756: cpu: Result refactoring

2017-01-26 Thread Jason Lowe-Power
> On Jan. 19, 2017, 10:18 p.m., Jason Lowe-Power wrote: > > Does converting this to a class have any impact on performance? > > Rekai Gonzalez Alberquilla wrote: > The honest reply is I do not know. The long reply is: I did differential > performance analysis between the base of the 5

Re: [gem5-dev] Review Request 3756: cpu: Result refactoring

2017-01-26 Thread Rekai Gonzalez Alberquilla
> On Jan. 19, 2017, 10:18 p.m., Jason Lowe-Power wrote: > > Does converting this to a class have any impact on performance? The honest reply is I do not know. The long reply is: I did differential performance analysis between the base of the 5 patches and each of the patches. The 2nd or 3rd

Re: [gem5-dev] Review Request 3756: cpu: Result refactoring

2017-01-26 Thread Rekai Gonzalez Alberquilla
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3756/ --- (Updated Jan. 26, 2017, 1:40 p.m.) Review request for Default. Repository: gem5

Re: [gem5-dev] Review Request 3756: cpu: Result refactoring

2017-01-19 Thread Jason Lowe-Power
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3756/#review9285 --- Does converting this to a class have any impact on performance?

Re: [gem5-dev] Review Request 3756: cpu: Result refactoring

2017-01-16 Thread Rekai Gonzalez Alberquilla
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3756/ --- (Updated Jan. 16, 2017, 11:52 a.m.) Review request for Default. Repository: gem5

[gem5-dev] Review Request 3756: cpu: Result refactoring

2016-12-09 Thread Rekai Gonzalez Alberquilla
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3756/ --- Review request for Default. Repository: gem5 Description --- Changeset