[gem5-dev] Change in gem5/gem5[develop]: arch-x86: Fix how MediaOps sets the size of its operands.

2021-08-13 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/49203 ) Change subject: arch-x86: Fix how MediaOps sets the size of its operands. .. arch-x86: Fix how MediaOps sets the size of its

[gem5-dev] Change in gem5/gem5[develop]: arch-x86: Use existing constants to simplify some code in operands.isa.

2021-08-13 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/49244 ) Change subject: arch-x86: Use existing constants to simplify some code in operands.isa. .. arch-x86: Use

[gem5-dev] Re: SCons SimObject() and m5.objects hierarchy

2021-08-13 Thread Gabe Black via gem5-dev
Ok, it sounds like listing out the contents of the SimObject files is not going to be a problem then, and I'll plan on doing that. *I* still like the idea of adding hierarchy (this is why python has packages, c++ has namespaces, almost all modern languages have similar features, etc), but

[gem5-dev] Change in gem5/gem5[develop]: misc: Add some changes to .git-blame-ignore-revs.

2021-08-13 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/49243 ) Change subject: misc: Add some changes to .git-blame-ignore-revs. .. misc: Add some changes to

[gem5-dev] Change in gem5/gem5[develop]: arch-arm: Fix style in utility.hh.

2021-08-13 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/49144 ) Change subject: arch-arm: Fix style in utility.hh. .. arch-arm: Fix style in utility.hh. Change-Id:

[gem5-dev] Change in gem5/gem5[develop]: arch-x86: Stop printing a source operand for the monitor instruction.

2021-08-13 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/49225 ) Change subject: arch-x86: Stop printing a source operand for the monitor instruction. .. arch-x86: Stop printing a source

[gem5-dev] Change in gem5/gem5[develop]: arch-x86: Stop printing a source register for CPUID.

2021-08-13 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/49224 ) Change subject: arch-x86: Stop printing a source register for CPUID. .. arch-x86: Stop printing a source register for CPUID.

[gem5-dev] Re: Running clang-format

2021-08-13 Thread Jason Lowe-Power via gem5-dev
Hey Matt, I forgot to include a couple of important pieces of information :). 1. There are places that we want to disable clang format. It would be impossible to find these places in one big formatting change. 2. The big change will cause huge headaches merging. The benefits of having a more

[gem5-dev] Re: Running clang-format

2021-08-13 Thread Jason Lowe-Power via gem5-dev
Hi all, I have an updated proposal. FYI: It looks like since the changes are marked as WIP they don't send emails when they are updated. For those of you who haven't seen the changesets, you can see the discussion (and some great ideas from others!) at the following links: Clang-format file:

[gem5-dev] Re: SCons SimObject() and m5.objects hierarchy

2021-08-13 Thread Jason Lowe-Power via gem5-dev
Hey Gabe, Some responses inline below. On Thu, Aug 12, 2021 at 9:37 PM Gabe Black via gem5-dev wrote: > Hey folks, here are some of my thoughts on the SimObject() mechanism in > SCons, and the structure of the m5.objects package. > > Right now, when you put a SimObject('foo.py') line in a

[gem5-dev] Re: plan to eliminate the "zero register" concept at the CPU level

2021-08-13 Thread Gabe Black via gem5-dev
Unfortunately it looks like that mechanism I mentioned (for predicating reads/writes) makes some big assumptions which are not generally correct, they just happened to be correct in the situation that feature was initially used. I think I'm going to try to replace that mechanism with one which

[gem5-dev] Change in gem5/gem5[develop]: arch-arm: Eliminate the INTREG_DUMMY integer register.

2021-08-13 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/49223 ) Change subject: arch-arm: Eliminate the INTREG_DUMMY integer register. .. arch-arm: Eliminate the INTREG_DUMMY

[gem5-dev] Change in gem5/gem5[develop]: arch-x86: Stop printing a source operand for the monitor instruction.

2021-08-13 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/49225 ) Change subject: arch-x86: Stop printing a source operand for the monitor instruction. .. arch-x86: Stop

[gem5-dev] Change in gem5/gem5[develop]: arch-x86: Stop printing a source register for CPUID.

2021-08-13 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/49224 ) Change subject: arch-x86: Stop printing a source register for CPUID. .. arch-x86: Stop printing a source register