Get gcc to check that we don't ignore return values of functions that can...

src/utils.h

Index: src/utils.h
===================================================================
--- src/utils.h	(revision 5365)
+++ src/utils.h	(working copy)
@@ -951,6 +951,13 @@
   return BitCast<Dest>(reinterpret_cast<uintptr_t>(source));
 }
 
+
+#ifdef __GNUC__

[Vyacheslav Egorov (Chromium), 2010/08/30 16:31:25]

1) GCC reference manual mentions this attribute starting from 3.4 so v8 will
fail to compile by previous versions of GCC.
2) there are some differences in __attribute__ syntax [position of __attribute__
modifier for GCC <4 and GCC >=4]. See INLINE macro.
3) Can we come up with more informative name? NOIGNORE is difficult to
understand. NOIGNORE what? 
4) Maybe move this definitions to globals.h alongside with INLINE macroses?

+#define NOIGNORE __attribute__ ((warn_unused_result))
+#else
+#define NOIGNORE
+#endif
+
 } }  // namespace v8::internal
 
 #endif  // V8_UTILS_H_