[PATCH] D78356: [flang] Replace ATTRIBUTE_UNUSED with LLVM_ATTRIBUTE_UNUSED.

مهدي شينون (Mehdi Chinoune) via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 30 10:08:32 PDT 2020


ChinouneMehdi updated this revision to Diff 261273.
ChinouneMehdi added a comment.

Put llvm/include first.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78356/new/

https://reviews.llvm.org/D78356

Files:
  flang/include/flang/Common/idioms.h


Index: flang/include/flang/Common/idioms.h
===================================================================
--- flang/include/flang/Common/idioms.h
+++ flang/include/flang/Common/idioms.h
@@ -13,6 +13,7 @@
 // or that is too weird or too specific to the host C++ compiler to be
 // exposed elsewhere.
 
+#include "llvm/Support/Compiler.h"
 #include <functional>
 #include <list>
 #include <optional>
@@ -98,10 +99,6 @@
   } \
   template <typename A> constexpr bool T{class_trait_ns_##T::trait_value<A>()};
 
-#if !defined ATTRIBUTE_UNUSED && (__clang__ || __GNUC__)
-#define ATTRIBUTE_UNUSED __attribute__((unused))
-#endif
-
 // Define enum class NAME with the given enumerators, a static
 // function EnumToString() that maps enumerators to std::string,
 // and a constant NAME_enumSize that captures the number of items
@@ -116,11 +113,11 @@
 
 #define ENUM_CLASS(NAME, ...) \
   enum class NAME { __VA_ARGS__ }; \
-  ATTRIBUTE_UNUSED static constexpr std::size_t NAME##_enumSize{[] { \
+  LLVM_ATTRIBUTE_UNUSED static constexpr std::size_t NAME##_enumSize{[] { \
     enum { __VA_ARGS__ }; \
     return Fortran::common::ListItemCount{__VA_ARGS__}.value; \
   }()}; \
-  ATTRIBUTE_UNUSED static inline std::string EnumToString(NAME e) { \
+  LLVM_ATTRIBUTE_UNUSED static inline std::string EnumToString(NAME e) { \
     return Fortran::common::EnumIndexToString( \
         static_cast<int>(e), #__VA_ARGS__); \
   }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D78356.261273.patch
Type: text/x-patch
Size: 1425 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200430/78d65314/attachment.bin>


More information about the llvm-commits mailing list