OpenCascade/occt
1
0
Fork 0
mirror of https://git.dev.opencascade.org/repos/occt.git synced 2025-08-14 13:30:48 +03:00
Code Activity

0028931: Eliminate dependency from TBB in OSD_Parallel header

Browse Source 
Implementation of methods OSD_Parallel::For() and ForEach() is moved to CXX files to avoid direct dependency of client code that uses OSD_Parallel on TBB headers, and necessity to link with TBB explicitly.
Runtime polymorphism (virtual methods) is used to hide implementation (TBB or threads-based).
This commit is contained in:
oan
2017-09-15 17:22:30 +03:00
committed by bugmaster
parent b92bc5ab7c
commit 00af0ebb9d
9 changed files with 471 additions and 256 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
src/QABugs/QABugs_19.cxx
View File

@@ -54,6 +54,7 @@
#include <XmlDrivers_DocumentRetrievalDriver.hxx>
#include <XmlDrivers_DocumentStorageDriver.hxx>
#include <TDataStd_Real.hxx>
#include <Standard_Atomic.hxx>
#include <cstdio>
#include <cmath>
@@ -2864,19 +2865,19 @@ struct OCC25545_Functor
//function : OCC25545
//purpose : Tests data race when concurrently accessing TopLoc_Location::Transformation()
//=======================================================================
#ifdef HAVE_TBB
static Standard_Integer OCC25545 (Draw_Interpretor& di,
Standard_Integer,
const char **)
{
// Place vertices in a vector, giving the i-th vertex the
// transformation that translates it on the vector (i,0,0) from the origin.
size_t n = 1000;
Standard_Integer n = 1000;
std::vector<TopoDS_Shape> aShapeVec (n);
std::vector<TopLoc_Location> aLocVec (n);
TopoDS_Shape aShape = BRepBuilderAPI_MakeVertex (gp::Origin ());
aShapeVec[0] = aShape;
for (size_t i = 1; i < n; ++i) {
for (Standard_Integer i = 1; i < n; ++i) {
gp_Trsf aT;
aT.SetTranslation (gp_Vec (1, 0, 0));
aLocVec[i] = aLocVec[i - 1] * aT;
@@ -2887,20 +2888,12 @@ static Standard_Integer OCC25545 (Draw_Interpretor& di,
// concurrently
OCC25545_Functor aFunc(aShapeVec);
//concurrently process
tbb::parallel_for (size_t (0), n, aFunc, tbb::simple_partitioner ());
// concurrently process
OSD_Parallel::For (0, n, aFunc);
QVERIFY (!aFunc.myIsRaceDetected);
return 0;
}
#else
static Standard_Integer OCC25545 (Draw_Interpretor&,
Standard_Integer,
const char **argv)
{
cout << "Test skipped: command " << argv[0] << " requires TBB library" << endl;
return 0;
}
#endif
//=======================================================================
//function : OCC25547