Reverts #278

CRAN advised that we revert this since the "Classic" intel compiler this was intended to support is deprecated and will be removed in 2023.

It also did not even really help the original issue (r-lib/systemfonts#98) since there were other compilation errors there.

Since this is such an old system and we have only ever had 1 report about this, I feel fine reverting it back.

Author: DavisVaughan

NEWS.md

@@ -43,9 +43,6 @@
 
 * Minor performance improvements to the cpp11 protect code. (@kevinushey)
 
-* Silenced an unknown attribute warning specific to the Intel compiler
-  (r-lib/systemfonts#98).
-
 * `cpp_register()` gains an argument `extension=` governing the file extension of
   the `src/cpp11` file. By default it's `.cpp`, but `.cc` is now supported 
   as well (#292, @MichaelChirico)

inst/include/cpp11/R.hpp

@@ -13,16 +13,12 @@
 #include "Rinternals.h"
 
 // clang-format off
-#if defined(__INTEL_COMPILER)
-# pragma warning(disable : 3924)
-#endif
-
-#if defined(__clang__)
+#ifdef __clang__
 # pragma clang diagnostic push
 # pragma clang diagnostic ignored "-Wattributes"
 #endif
 
-#if defined(__GNUC__) && !defined(__clang__) && !defined(__INTEL_COMPILER)
+#ifdef __GNUC__
 # pragma GCC diagnostic push
 # pragma GCC diagnostic ignored "-Wattributes"
 #endif