aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authoreschnett <eschnett@105869f7-3296-0410-a4ea-f4349344b45a>2013-05-06 18:28:13 +0000
committereschnett <eschnett@105869f7-3296-0410-a4ea-f4349344b45a>2013-05-06 18:28:13 +0000
commit825b89e0e6bf3e4e248188b36f5b29029737d44a (patch)
tree66b804d03385b1b9cdf0f2eb3c3b9aaf3ca8bf5e
parent89b9df75a65eebe20cff4d03785db434bbb4c711 (diff)
Update comment explaining Intel compiler bug
git-svn-id: https://svn.cct.lsu.edu/repos/numrel/LSUThorns/Vectors/trunk@82 105869f7-3296-0410-a4ea-f4349344b45a
-rw-r--r--src/vectors-8-SSE2.h4
1 files changed, 3 insertions, 1 deletions
diff --git a/src/vectors-8-SSE2.h b/src/vectors-8-SSE2.h
index 2326e49..bded69e 100644
--- a/src/vectors-8-SSE2.h
+++ b/src/vectors-8-SSE2.h
@@ -185,7 +185,9 @@ CCTK_REAL8_VEC vec8_loadu_maybe3(std::ptrdiff_t const off1,
static inline CCTK_ATTRIBUTE_ALWAYS_INLINE
CCTK_REAL8_VEC vec8_loadu_maybe(std::ptrdiff_t const off, CCTK_REAL8 const& p)
{
- // The :? operator breaks with the Intel compiler
+ // Note: This function is mis-translated by some versions of the
+ // Intel compiler. The symptom is a segfault during the Vectors
+ // selftest in the vec_loadu_maybe test.
// return off % CCTK_REAL8_VEC_SIZE == 0 ? vec8_load(p) : vec8_load_off1(p);
if (off % CCTK_REAL8_VEC_SIZE == 0) return vec8_load(p);
return vec8_load_off1(p);