David Ribeiro Alves has uploaded a new change for review.
http://gerrit.cloudera.org:8080/6693
Change subject: Silence clang -Waddress-of-packed-member warning
......................................................................
Silence clang -Waddress-of-packed-member warning
macOS builds have become polluted with warnings like:
In file included from ../../src/kudu/tablet/deltamemstore.cc:22:
In file included from ../../src/kudu/tablet/deltafile.h:34:
In file included from ../../src/kudu/tablet/deltamemstore.h:33:
../../src/kudu/tablet/concurrent_btree.h:386:33: warning: taking address of
packed member 'version_' of class or structure
'kudu::tablet::btree::NodeBase<kudu::tablet::DMSTreeTraits>' may result in an
unaligned pointer value [-Waddress-of-packed-member]
VersionField::SetInserting(&version_);
^~~~~~~~
../../src/kudu/tablet/concurrent_btree.h:721:11: note: in instantiation of
member function
'kudu::tablet::btree::NodeBase<kudu::tablet::DMSTreeTraits>::SetInserting'
requested here
this->SetInserting();
^
../../src/kudu/tablet/concurrent_btree.h:707:12: note: in instantiation of
member function
'kudu::tablet::btree::LeafNode<kudu::tablet::DMSTreeTraits>::InsertNew'
requested here
return InsertNew(mut->idx(), mut->key(), val, mut->arena());
^
../../src/kudu/tablet/concurrent_btree.h:1316:19: note: in instantiation of
member function
'kudu::tablet::btree::LeafNode<kudu::tablet::DMSTreeTraits>::Insert' requested
here
switch (node->Insert(mutation, val)) {
^
../../src/kudu/tablet/concurrent_btree.h:869:19: note: in instantiation of
member function
'kudu::tablet::btree::CBTree<kudu::tablet::DMSTreeTraits>::Insert' requested
here
return tree_->Insert(this, val);
^
../../src/kudu/tablet/deltamemstore.cc:103:31: note: in instantiation of member
function
'kudu::tablet::btree::PreparedMutation<kudu::tablet::DMSTreeTraits>::Insert'
requested here
if (PREDICT_FALSE(!mutation.Insert(update.slice()))) {
This is because of clang's -Waddress-of-packed-member warning which has
been committed and reverted and committed again in the clang codebase.
Chromium also silenced this (multiple times), see:
https://bugs.chromium.org/p/chromium/issues/detail?id=637306
Change-Id: I03518bcca43ea6c891a7f622972465c1ea87ce63
---
M CMakeLists.txt
1 file changed, 5 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/93/6693/1
--
To view, visit http://gerrit.cloudera.org:8080/6693
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I03518bcca43ea6c891a7f622972465c1ea87ce63
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <[email protected]>
