Bert Wesarg
Bert Wesarg
I'm now using this: ```diff @@ -141,16 +141,16 @@ struct aligned_chunk { long long x; }; template -struct aligned_chunk { - __attribute__((aligned(16))) char x; +struct alignas(16) aligned_chunk { + char...
These are my current warnings without the patch: ``` mpicc -c -o dart-impl/base/src/internal/domain_locality.o -DSPEC -DNDEBUG -Iinclude -DBOOST_ALL_NO_LIB -Idart-impl/mpi/include -Idart-impl/mpi/src -Idart-impl/base/include -DDART_HAVE_SYNC_BUILTINS -DDART -O3 -march=native -DDASH_HAVE_STD_TRIVIALLY_COPYABLE -DDART_MPI_DISABLE_SHARED_WINDOWS dart-impl/base/src/internal/domain_locality.c dart-impl/base/src/internal/domain_locality.c: In function...
Obviously none, because that are two different files ;-)
Could you test it with Intel 19? I could send this one-liner to the original report, but it may take some time to get feedback.
BTW, I'm still not able to look into the build logs from circle CI. I'm just do not see the uploaded `build.log` artifact. If there is anything others need to...
As you disable all of the traversial API, than why do we have this `_children` member at all? I think most users can live without this member. AFAIK, MPI does...
There is one reason though, the deconstructor, which also removes all children. I'm fine with this as is.
I updated the description with a working reproducer.
@davidism any comments?