From 8130bd7a2405dbf889b8a0e3baa54ace7b630afc Mon Sep 17 00:00:00 2001 From: Jeffrey Walton Date: Mon, 19 Aug 2019 08:30:11 -0400 Subject: [PATCH] Update comments --- darn.h | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/darn.h b/darn.h index 91a22d0ef..a2164d9cc 100644 --- a/darn.h +++ b/darn.h @@ -2,11 +2,12 @@ // DARN requires POWER9/ISA 3.0. // At the moment only GCC 7.0 (and above) seems to support __builtin_darn() -// and __builtin_darn_32(). Clang 7.0 does not provide them, but it does -// support assembly instructions. XLC is unknown, but there are no hits when -// searching IBM's site. To cover more platforms we provide GCC inline -// assembly like we do with RDRAND and RDSEED. Platforms that don't support -// GCC inline assembly or the builtin will fail the compile. Also see +// and __builtin_darn_32(). However, GCC generates incorrect code. Clang 7.0 +// does not provide them, but it does support assembly instructions. XLC is +// unknown, but there are no hits when searching IBM's site. To cover more +// platforms we provide GCC inline assembly like we do with RDRAND and RDSEED. +// Platforms that don't support GCC inline assembly or the builtin will fail +// to compile. Also see https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91481 and // https://gcc.gnu.org/onlinedocs/gcc/Basic-PowerPC-Built-in-Functions-Available-on-ISA-3_002e0.html /// \file darn.h