Hi all,

My Rogue7 dashboard has a few warnings from clang's -Wover-aligned:

<http://open.cdash.org/viewBuildError.php?type=1&buildid=2911702>

I'm not a language lawyer, but these look legit to me.  Since I build as C++11 
I fall into:

#define ITK_CACHE_LINE_ALIGNMENT 64

# define itkAlignedTypedef( alignment, oldtype, newtype )   \
  typedef oldtype newtype alignas(alignment)

The standard says this about operator new: "The pointer returned shall be 
suitably aligned so that it can be converted to a pointer of any complete 
object type with a fundamental alignment requirement."  std::max_align_t has 
the strictest fundamental alignment, and, on my system anyway, it's smaller 
than 64.

Is ITK_CACHE_LINE_ALIGNMENT only for performance?  Maybe it should be removed?

Thanks,

-- 
____________________________________________________________
Sean McBride, B. Eng                 [email protected]
Rogue Research                        www.rogue-research.com 
Mac Software Developer              Montréal, Québec, Canada


_______________________________________________
Powered by www.kitware.com

Visit other Kitware open-source projects at
http://www.kitware.com/opensource/opensource.html

Kitware offers ITK Training Courses, for more information visit:
http://kitware.com/products/protraining.php

Please keep messages on-topic and check the ITK FAQ at:
http://www.itk.org/Wiki/ITK_FAQ

Follow this link to subscribe/unsubscribe:
http://www.itk.org/mailman/listinfo/insight-developers

Reply via email to