Skip to content

[tbb] fix tbb by updating it#23567

Closed
Neumann-A wants to merge 19 commits intomicrosoft:masterfrom
Neumann-A:fix_tbb_linux
Closed

[tbb] fix tbb by updating it#23567
Neumann-A wants to merge 19 commits intomicrosoft:masterfrom
Neumann-A:fix_tbb_linux

Conversation

@Neumann-A
Copy link
Copy Markdown
Contributor

No description provided.

Alexander Neumann added 2 commits March 15, 2022 16:22
@mathisloge
Copy link
Copy Markdown
Contributor

related #19909 and #23058

@Neumann-A Neumann-A marked this pull request as draft March 15, 2022 21:39
@JonLiu1993 JonLiu1993 added the category:port-update The issue is with a library, which is requesting update new revision label Mar 16, 2022
Comment on lines +4 to 5
REF ea956014ed45906dda21e1ee682d297189cb4a7b
SHA512 57638cd0b23b5f2bfd32fdc159d6cd77ca34e3bd695c225591979aef4b7271eac93d3706fa1ffa2340f90013267a4171bebe1e4c142f19ad2bf67963dfed627e
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm why isn't CI choking on this? Is the hash check broken somehow? cc @BillyONeal @strega-nil

@JackBoosY
Copy link
Copy Markdown
Contributor

See why tbb is not updated in #16321.

@Neumann-A
Copy link
Copy Markdown
Contributor Author

@JackBoosY: the only one blocking this is the usd port and ras0219 put in on the skip list in #19909

@Neumann-A
Copy link
Copy Markdown
Contributor Author

Gazebo error:

D:\buildtrees\gazebo\src\7189cb4a7b-e170e5c60c.clean\gazebo\transport\Connection.hh(58): error C2516: 'tbb::v1::task': is not a legal base class
D:\installed\x86-windows\include\tbb\../oneapi/tbb/task.h(26): note: see declaration of 'tbb::v1::task'
D:\buildtrees\gazebo\src\7189cb4a7b-e170e5c60c.clean\gazebo\transport\Connection.hh(73): error C2143: syntax error: missing ';' before '*'
D:\buildtrees\gazebo\src\7189cb4a7b-e170e5c60c.clean\gazebo\transport\Connection.hh(73): error C2886: 'tbb::v1::task': symbol cannot be used in a member using-declaration
D:\installed\x86-windows\include\tbb\../oneapi/tbb/task.h(26): note: see declaration of 'tbb::v1::task'
D:\buildtrees\gazebo\src\7189cb4a7b-e170e5c60c.clean\gazebo\transport\Connection.hh(74): error C2334: unexpected token(s) preceding '{'; skipping apparent function body
D:\buildtrees\gazebo\src\7189cb4a7b-e170e5c60c.clean\gazebo\transport\Connection.hh(313): error C2039: 'allocate_root': is not a member of 'tbb::v1::task'
D:\installed\x86-windows\include\tbb\../oneapi/tbb/task.h(26): note: see declaration of 'tbb::v1::task'
D:\buildtrees\gazebo\src\7189cb4a7b-e170e5c60c.clean\gazebo\transport\Connection.hh(294): note: This diagnostic occurred in the compiler generated function 'void gazebo::transport::Connection::OnReadData(const boost::system::error_code &,boost::tuples::tuple<Handler,boost::tuples::null_type,boost::tuples::null_type,boost::tuples::null_type,boost::tuples::null_type,boost::tuples::null_type,boost::tuples::null_type,boost::tuples::null_type,boost::tuples::null_type,boost::tuples::null_type>)'
D:\buildtrees\gazebo\src\7189cb4a7b-e170e5c60c.clean\gazebo\transport\Connection.hh(315): error C2039: 'enqueue': is not a member of 'tbb::v1::task'
D:\installed\x86-windows\include\tbb\../oneapi/tbb/task.h(26): note: see declaration of 'tbb::v1::task'
D:\buildtrees\gazebo\src\7189cb4a7b-e170e5c60c.clean\gazebo\transport\Connection.hh(294): note: This diagnostic occurred in the compiler generated function 'void gazebo::transport::Connection::OnReadData(const boost::system::error_code &,boost::tuples::tuple<Handler,boost::tuples::null_type,boost::tuples::null_type,boost::tuples::null_type,boost::tuples::null_type,boost::tuples::null_type,boost::tuples::null_type,boost::tuples::null_type,boost::tuples::null_type,boost::tuples::null_type>)'
D:\buildtrees\gazebo\src\7189cb4a7b-e170e5c60c.clean\gazebo/transport/Node.hh(40): error C2516: 'tbb::v1::task': is not a legal base class
D:\installed\x86-windows\include\tbb\../oneapi/tbb/task.h(26): note: see declaration of 'tbb::v1::task'
D:\buildtrees\gazebo\src\7189cb4a7b-e170e5c60c.clean\gazebo/transport/Node.hh(54): error C2143: syntax error: missing ';' before '*'
D:\buildtrees\gazebo\src\7189cb4a7b-e170e5c60c.clean\gazebo/transport/Node.hh(54): error C2886: 'tbb::v1::task': symbol cannot be used in a member using-declaration
D:\installed\x86-windows\include\tbb\../oneapi/tbb/task.h(26): note: see declaration of 'tbb::v1::task'
D:\buildtrees\gazebo\src\7189cb4a7b-e170e5c60c.clean\gazebo/transport/Node.hh(55): error C2334: unexpected token(s) preceding '{'; skipping apparent function body
D:\buildtrees\gazebo\src\7189cb4a7b-e170e5c60c.clean\gazebo/transport/Node.hh(162): error C2039: 'allocate_root': is not a member of 'tbb::v1::task'
D:\installed\x86-windows\include\tbb\../oneapi/tbb/task.h(26): note: see declaration of 'tbb::v1::task'
D:\buildtrees\gazebo\src\7189cb4a7b-e170e5c60c.clean\gazebo/transport/Node.hh(158): note: This diagnostic occurred in the compiler generated function 'void gazebo::transport::Node::Publish(const std::string &,const google::protobuf::Message &)'
D:\buildtrees\gazebo\src\7189cb4a7b-e170e5c60c.clean\gazebo/transport/Node.hh(165): error C2039: 'enqueue': is not a member of 'tbb::v1::task'
D:\installed\x86-windows\include\tbb\../oneapi/tbb/task.h(26): note: see declaration of 'tbb::v1::task'
D:\buildtrees\gazebo\src\7189cb4a7b-e170e5c60c.clean\gazebo/transport/Node.hh(158): note: This diagnostic occurred in the compiler generated function 'void gazebo::transport::Node::Publish(const std::string &,const google::protobuf::Message &)'

@Neumann-A
Copy link
Copy Markdown
Contributor Author

pagmo2 error:

FAILED: CMakeFiles/pagmo.dir/src/algorithms/mbh.cpp.obj 
D:\installed\x86-windows\include\boost/mpl/print.hpp(55): error C2220: the following warning is treated as an error
... long template chain
D:\installed\x86-windows\include\boost/mpl/print.hpp(55): warning C4308: negative integral constant converted to unsigned type

@JackBoosY
Copy link
Copy Markdown
Contributor

Can I help you to fix these regressions?

@Neumann-A
Copy link
Copy Markdown
Contributor Author

Can I help you to fix these regressions?

@JackBoosY: You can try. I won't be able to do anything here until the weekend probably,.

@JackBoosY JackBoosY changed the title [baseline] fix tbb by updating it [tbb] fix tbb by updating it Mar 21, 2022
@Thomas1664
Copy link
Copy Markdown
Contributor

pagmo2 error:

FAILED: CMakeFiles/pagmo.dir/src/algorithms/mbh.cpp.obj 
D:\installed\x86-windows\include\boost/mpl/print.hpp(55): error C2220: the following warning is treated as an error
... long template chain
D:\installed\x86-windows\include\boost/mpl/print.hpp(55): warning C4308: negative integral constant converted to unsigned type

This can be fixed by patching out -Werror or /WX

@Tradias
Copy link
Copy Markdown
Contributor

Tradias commented Apr 26, 2022

I already wrote the -Werror patch in ras0219-msft's pull request: #19909 (comment)

@JackBoosY
Copy link
Copy Markdown
Contributor

JackBoosY commented Jul 15, 2022

Any progress here? Ignore me.

@JackBoosY JackBoosY added the depends:different-pr This PR or Issue depends on a PR which has been filed label Jul 15, 2022
@Neumann-A
Copy link
Copy Markdown
Contributor Author

superseded by #26284

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-update The issue is with a library, which is requesting update new revision depends:different-pr This PR or Issue depends on a PR which has been filed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants