No, not that I'm aware of. It would just be to make it feasible to
implement the warn_once functionality without using macros. With c++20, I
can more or less get it to work with some minor template syntax,
warn("xyz"), but that relies on the source location (file, line,
column which may be iffy)
Or I should say without un-namespaced macros (GEM5_ prefixed), since
GEM5_ONCE itself would be a macro.
Gabe
On Mon, Jul 26, 2021 at 11:01 PM Gabe Black wrote:
> No, not that I'm aware of. It would just be to make it feasible to
> implement the warn_once functionality without using macros.
Gabe Black has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/46139 )
Change subject: fastmodel: Minimally implement reading MiscRegs for the
CortexR52.
..
fastmodel: Minimally implement
Gabe Black has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/48603 )
Change subject: dev: Drop include of drm/drm.h in kfd_ioctl.h.
..
dev: Drop include of drm/drm.h in kfd_ioctl.h.
I don't
Gabe Black has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/48647 )
Change subject: mem: Use the new "debug" namespace, and not "Debug".
..
mem: Use the new "debug" namespace, and not "Debug".
Gabe Black has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/48512 )
Change subject: scons: Increase the minimum version of gcc to 7.
..
scons: Increase the minimum version of gcc to 7.
This
Yu-hsin Wang has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/45581 )
Change subject: fastmodel: Use Iris API to access memory
..
fastmodel: Use Iris API to access memory
Memory space is not
Yeah, that's not a bad option either. Between the two, I suggested
GEM5_ONCE so that the _once version wouldn't be as inconsistent with the
normal version of warn, and even though the GEM5_ONCE macro probably
wouldn't be useful outside of warn, it's a pretty generic mechanism and
could be. For
Gabe Black has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/48368 )
Change subject: scons,test: Don't -Wall or -Werror for googletest libraries.
..
scons,test: Don't -Wall or -Werror for
Gabe Black has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/48644 )
Change subject: scons: Get rid of some redundant text in a print out.
..
scons: Get rid of some redundant text in a print
Gabe Black has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/48645 )
Change subject: scons: Add the "print_suppressions=0" option to the LSAN
warning.
..
scons: Add the
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/48643 )
Change subject: util: Make --with-asan work with more versions of the
python lib.
..
util: Make --with-asan
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/48647 )
Change subject: mem: Use the new "debug" namespace, and not "Debug".
..
mem: Use the new "debug" namespace, and
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/48645 )
Change subject: scons: Add the "print_suppressions=0" option to the LSAN
warning.
..
scons: Add the
This is great, thanks Gabe!
> -Original Message-
> From: Gabe Black via gem5-dev
> Sent: 25 July 2021 12:45
> To: gem5 Developer List ; Earl Ou
>
> Cc: Gabe Black
> Subject: [gem5-dev] "epic" Jira issue to track multi-ISA gem5
>
> To help lay out a roadmap and scope for my ongoing
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/48646 )
Change subject: scons: Disable the free-nonheap-object warning for gcc.
..
scons: Disable the free-nonheap-object
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/48644 )
Change subject: scons: Get rid of some redundant text in a print out.
..
scons: Get rid of some redundant text in
Hi folks, specifically ARM folks. I'm trying to figure out what's going on
with this in the minor CPU:
#if THE_ISA == X86_ISA || THE_ISA == ARM_ISA
/* In SE mode, it's possible to branch to a microop when
* replaying faults such as page faults (or simply
Bobby R. Bruce has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/48586 )
Change subject: util: Fix gerrit bot commit subject parser
..
util: Fix gerrit bot commit subject parser
Currently, if
Gabe Black has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/48513 )
Change subject: scons: Update the minimum version of python to 3.6.
..
scons: Update the minimum version of python to 3.6.
Gabe Black has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/48643 )
Change subject: util: Make --with-asan work with more versions of the
python lib.
..
util: Make --with-asan work with
Wouldn't we get the same net result by just renaming warn_once() to
GEM5_WARN_ONCE()? Seems simpler .
Steve
On Mon, Jul 26, 2021 at 11:03 PM Gabe Black wrote:
> Or I should say without un-namespaced macros (GEM5_ prefixed), since
> GEM5_ONCE itself would be a macro.
>
> Gabe
>
> On Mon, Jul
This will make things so much better. I very much support it.
Thanks for creating the Jira epic for tracking this.
--
Dr. Bobby R. Bruce
Room 3050,
Kemper Hall, UC Davis
Davis,
CA, 95616
web: https://www.bobbybruce.net
On Tue, Jul 27, 2021 at 3:42 AM Giacomo Travaglini via gem5-dev <
Bobby R. Bruce has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/48583 )
Change subject: misc: Update the version to v21.1.0.0
..
misc: Update the version to v21.1.0.0
Change-Id:
Bobby R. Bruce has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/48584 )
Change subject: scons: Remove -Werror for the gem5 21.1 release
..
scons: Remove -Werror for the gem5 21.1 release
Bobby R. Bruce has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/48663 )
Change subject: docker-util: Update the gcn-gpu docker image Cloud Bucket
..
docker-util: Update the gcn-gpu
Hello,
I like the idea of GEM5_ONCE, but as it would become part of the API users
could definitely use in unforeseen ways (e.g., debugging). I am not saying this
is a bad thing, but it is something to consider.
Regards,Daniel
___
gem5-dev mailing list
27 matches
Mail list logo