base,dev: Use the standard attribute [[noreturn]].
authorGabe Black <gabeblack@google.com>
Sat, 26 Sep 2020 23:33:28 +0000 (16:33 -0700)
committerGabe Black <gabeblack@google.com>
Mon, 28 Sep 2020 05:41:45 +0000 (05:41 +0000)
The [[noreturn]] attribute has been standard since c++11, and so we
don't (and haven't for a while) need to wrap it in a macro.

Change-Id: Ifba62c87c19224bb366e93ebba685a063cc750ce
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/35218
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/base/compiler.hh
src/base/logging.hh
src/dev/arm/gpu_nomali.hh

index c3b2510a989562e12bc3cfaf654416b349abda8b..c2eb5d910f76afe496ea680ebe7e6be3bbd45aeb 100644 (file)
@@ -46,7 +46,6 @@
 // http://gcc.gnu.org/onlinedocs/gcc/Function-Attributes.html
 
 #if defined(__GNUC__) // clang or gcc
-#  define M5_ATTR_NORETURN  __attribute__((noreturn))
 #  define M5_VAR_USED __attribute__((unused))
 #  define M5_ATTR_PACKED __attribute__ ((__packed__))
 #  define M5_NO_INLINE __attribute__ ((__noinline__))
index f56420bfa6621f40848605e3a385c1889523e1c6..29a9563d2b07656076b3d3b99d0e9ce1fd5bbc7a 100644 (file)
@@ -121,7 +121,7 @@ class Logger
      * functions, and gcc will get mad if a function calls panic and then
      * doesn't return.
      */
-    void exit_helper() M5_ATTR_NORETURN { exit(); ::abort(); }
+    [[noreturn]] void exit_helper() { exit(); ::abort(); }
 
   protected:
     bool enabled;
index 1880ec6c34e4adbefdd5cde8566a822b78cfa031..8c3ac265da50093e5347b988e38dc763bff16d6d 100644 (file)
@@ -99,7 +99,7 @@ class NoMaliGpu : public PioDevice
      * @param err Error code from the NoMali library.
      * @param msg Message to print.
      */
-    static void gpuPanic(nomali_error_t err, const char *msg) M5_ATTR_NORETURN;
+    [[noreturn]] static void gpuPanic(nomali_error_t err, const char *msg);
     /**
      * Panic if the NoMali returned an error, do nothing otherwise.
      *