Re: [Paraview] paraview git build with pgi 12.4 errors
Thanks. I will push the patch to gerrit tonight. Best, George On Mon, Aug 20, 2012 at 12:40 PM, Utkarsh Ayachit wrote: > Thanks Mark. > > George, can you please get the vtkExtentRCBPartitioner fix in to VTK? thanks. > > Utkarsh > > On Mon, Aug 20, 2012 at 12:13 PM, Vanmoer, Mark W > wrote: >> Hi Utkarsh, >> >> The patch worked, thank you. I hadn't seen that before so I made a small >> test and both Intel and GNU will compile code that uses ++ with an iterator, >> but PGI won't. >> >> I also had: >> >> VTK/Common/ExecutionModel/vtkExtentRCBPartitioner.h", line 160: error: >> namespace "std" has no member "string" >>void PrintExtent( std::string name, int ext[6] ); >> >> which I fixed with #include as the AMR Enzo reader below. >> >> Also got these errors, but I don't need this plugin so I just disabled it. >> Looks like PGI doesn't support those _mm_* functions. >> >> >> "/usr/apps/vis/ParaView/repo/ParaView/Plugins/SciberQuestToolKit/eigen-3.0.3/eigen-eigen-3.0.3/Eigen/src/Core/util/Memory.h", >> line 209: error: >> identifier "_mm_malloc" is undefined >> result = _mm_malloc(size, 16); >>^ >> >> "/usr/apps/vis/ParaView/repo/ParaView/Plugins/SciberQuestToolKit/eigen-3.0.3/eigen-eigen-3.0.3/Eigen/src/Core/util/Memory.h", >> line 233: error: >> identifier "_mm_free" is undefined >> _mm_free(ptr); >> ^ >> >> Thanks, >> Mark >> -Original Message- >> From: Utkarsh Ayachit [mailto:utkarsh.ayac...@kitware.com] >> Sent: Friday, August 17, 2012 12:53 PM >> To: Vanmoer, Mark W >> Cc: paraview@paraview.org >> Subject: Re: [Paraview] paraview git build with pgi 12.4 errors >> >> Mark, >> >> Can you try the attached patch? It should resolve the lvalue errors. >> >> Thanks >> Utkarsh >> >> On Fri, Aug 17, 2012 at 12:40 PM, Vanmoer, Mark W >> wrote: >>> Hello, I'm attempting to build ParaView from the git repo with PGI >>> 12.4 and I get the following errors for vtkSMTestDriver.cxx that I >>> don't know how to >>> fix: >>> >>> >>> >>> "/usr/apps/vis/ParaView/repo/ParaView/Utilities/TestDriver/vtkSMTestDr >>> iver.cxx", >>> line 434: error: >>> >>> expression must be a modifiable lvalue >>> >>> commandLine.insert(++commandLine.begin(), "--multi-servers"); >>> >>>^ >>> >>> >>> >>> "/usr/apps/vis/ParaView/repo/ParaView/Utilities/TestDriver/vtkSMTestDr >>> iver.cxx", >>> line 1296: error: >>> >>> expression must be a modifiable lvalue >>> >>> clientCommand.insert(++clientCommand.begin(), >>> this->ServerURL.c_str()); >>> >>>^ >>> >>> >>> >>> Also these, which I got around by adding #include >>> >>> "/usr/apps/vis/ParaView/repo/ParaView/ParaViewCore/ClientServerCore/Co >>> re/vtkProcessModule.cxx", >>> line 57: error: >>> >>> identifier "LC_NUMERIC" is undefined >>> >>> setlocale(LC_NUMERIC,"C"); >>> >>> ^ >>> >>> >>> >>> "/usr/apps/vis/ParaView/repo/ParaView/ParaViewCore/ClientServerCore/Co >>> re/vtkProcessModule.cxx", >>> line 57: error: >>> >>> identifier "setlocale" is undefined >>> >>> setlocale(LC_NUMERIC,"C"); >>> >>> ^ >>> >>> >>> >>> And this, which I got around by adding #include >>> >>> >>> >>> "/usr/apps/vis/ParaView/repo/ParaView/VTK/Filters/AMR/vtkAMREnzoReader >>> .h", >>> line 70: error: >>> >>> namespace "std" has no member "string" >>> >>> int GetIndexFromArrayName( std::string arrayName ); >>> >>> >>> >>> >>> >>> Can someone suggest a fix for the lvalue error? >>> >>> Thanks, >>> >>> Mark >>> >>> >>> >>> >>> ___ >>> Powered by www.kitware.com >>> >>> Visit other Kitware open-source projects at >>> http://www.kitware.com/opensource/opensource.html >>> >>> Please keep messages on-topic and check the ParaView Wiki at: >>> http://paraview.org/Wiki/ParaView >>> >>> Follow this link to subscribe/unsubscribe: >>> http://www.paraview.org/mailman/listinfo/paraview >>> ___ Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Please keep messages on-topic and check the ParaView Wiki at: http://paraview.org/Wiki/ParaView Follow this link to subscribe/unsubscribe: http://www.paraview.org/mailman/listinfo/paraview
Re: [Paraview] paraview git build with pgi 12.4 errors
Thanks Mark. George, can you please get the vtkExtentRCBPartitioner fix in to VTK? thanks. Utkarsh On Mon, Aug 20, 2012 at 12:13 PM, Vanmoer, Mark W wrote: > Hi Utkarsh, > > The patch worked, thank you. I hadn't seen that before so I made a small test > and both Intel and GNU will compile code that uses ++ with an iterator, but > PGI won't. > > I also had: > > VTK/Common/ExecutionModel/vtkExtentRCBPartitioner.h", line 160: error: > namespace "std" has no member "string" >void PrintExtent( std::string name, int ext[6] ); > > which I fixed with #include as the AMR Enzo reader below. > > Also got these errors, but I don't need this plugin so I just disabled it. > Looks like PGI doesn't support those _mm_* functions. > > > "/usr/apps/vis/ParaView/repo/ParaView/Plugins/SciberQuestToolKit/eigen-3.0.3/eigen-eigen-3.0.3/Eigen/src/Core/util/Memory.h", > line 209: error: > identifier "_mm_malloc" is undefined > result = _mm_malloc(size, 16); >^ > > "/usr/apps/vis/ParaView/repo/ParaView/Plugins/SciberQuestToolKit/eigen-3.0.3/eigen-eigen-3.0.3/Eigen/src/Core/util/Memory.h", > line 233: error: > identifier "_mm_free" is undefined > _mm_free(ptr); > ^ > > Thanks, > Mark > -Original Message- > From: Utkarsh Ayachit [mailto:utkarsh.ayac...@kitware.com] > Sent: Friday, August 17, 2012 12:53 PM > To: Vanmoer, Mark W > Cc: paraview@paraview.org > Subject: Re: [Paraview] paraview git build with pgi 12.4 errors > > Mark, > > Can you try the attached patch? It should resolve the lvalue errors. > > Thanks > Utkarsh > > On Fri, Aug 17, 2012 at 12:40 PM, Vanmoer, Mark W > wrote: >> Hello, I'm attempting to build ParaView from the git repo with PGI >> 12.4 and I get the following errors for vtkSMTestDriver.cxx that I >> don't know how to >> fix: >> >> >> >> "/usr/apps/vis/ParaView/repo/ParaView/Utilities/TestDriver/vtkSMTestDr >> iver.cxx", >> line 434: error: >> >> expression must be a modifiable lvalue >> >> commandLine.insert(++commandLine.begin(), "--multi-servers"); >> >>^ >> >> >> >> "/usr/apps/vis/ParaView/repo/ParaView/Utilities/TestDriver/vtkSMTestDr >> iver.cxx", >> line 1296: error: >> >> expression must be a modifiable lvalue >> >> clientCommand.insert(++clientCommand.begin(), >> this->ServerURL.c_str()); >> >>^ >> >> >> >> Also these, which I got around by adding #include >> >> "/usr/apps/vis/ParaView/repo/ParaView/ParaViewCore/ClientServerCore/Co >> re/vtkProcessModule.cxx", >> line 57: error: >> >> identifier "LC_NUMERIC" is undefined >> >> setlocale(LC_NUMERIC,"C"); >> >> ^ >> >> >> >> "/usr/apps/vis/ParaView/repo/ParaView/ParaViewCore/ClientServerCore/Co >> re/vtkProcessModule.cxx", >> line 57: error: >> >> identifier "setlocale" is undefined >> >> setlocale(LC_NUMERIC,"C"); >> >> ^ >> >> >> >> And this, which I got around by adding #include >> >> >> >> "/usr/apps/vis/ParaView/repo/ParaView/VTK/Filters/AMR/vtkAMREnzoReader >> .h", >> line 70: error: >> >> namespace "std" has no member "string" >> >> int GetIndexFromArrayName( std::string arrayName ); >> >> >> >> >> >> Can someone suggest a fix for the lvalue error? >> >> Thanks, >> >> Mark >> >> >> >> >> ___ >> Powered by www.kitware.com >> >> Visit other Kitware open-source projects at >> http://www.kitware.com/opensource/opensource.html >> >> Please keep messages on-topic and check the ParaView Wiki at: >> http://paraview.org/Wiki/ParaView >> >> Follow this link to subscribe/unsubscribe: >> http://www.paraview.org/mailman/listinfo/paraview >> ___ Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Please keep messages on-topic and check the ParaView Wiki at: http://paraview.org/Wiki/ParaView Follow this link to subscribe/unsubscribe: http://www.paraview.org/mailman/listinfo/paraview
Re: [Paraview] paraview git build with pgi 12.4 errors
Hi Utkarsh, The patch worked, thank you. I hadn't seen that before so I made a small test and both Intel and GNU will compile code that uses ++ with an iterator, but PGI won't. I also had: VTK/Common/ExecutionModel/vtkExtentRCBPartitioner.h", line 160: error: namespace "std" has no member "string" void PrintExtent( std::string name, int ext[6] ); which I fixed with #include as the AMR Enzo reader below. Also got these errors, but I don't need this plugin so I just disabled it. Looks like PGI doesn't support those _mm_* functions. "/usr/apps/vis/ParaView/repo/ParaView/Plugins/SciberQuestToolKit/eigen-3.0.3/eigen-eigen-3.0.3/Eigen/src/Core/util/Memory.h", line 209: error: identifier "_mm_malloc" is undefined result = _mm_malloc(size, 16); ^ "/usr/apps/vis/ParaView/repo/ParaView/Plugins/SciberQuestToolKit/eigen-3.0.3/eigen-eigen-3.0.3/Eigen/src/Core/util/Memory.h", line 233: error: identifier "_mm_free" is undefined _mm_free(ptr); ^ Thanks, Mark -Original Message- From: Utkarsh Ayachit [mailto:utkarsh.ayac...@kitware.com] Sent: Friday, August 17, 2012 12:53 PM To: Vanmoer, Mark W Cc: paraview@paraview.org Subject: Re: [Paraview] paraview git build with pgi 12.4 errors Mark, Can you try the attached patch? It should resolve the lvalue errors. Thanks Utkarsh On Fri, Aug 17, 2012 at 12:40 PM, Vanmoer, Mark W wrote: > Hello, I'm attempting to build ParaView from the git repo with PGI > 12.4 and I get the following errors for vtkSMTestDriver.cxx that I > don't know how to > fix: > > > > "/usr/apps/vis/ParaView/repo/ParaView/Utilities/TestDriver/vtkSMTestDr > iver.cxx", > line 434: error: > > expression must be a modifiable lvalue > > commandLine.insert(++commandLine.begin(), "--multi-servers"); > >^ > > > > "/usr/apps/vis/ParaView/repo/ParaView/Utilities/TestDriver/vtkSMTestDr > iver.cxx", > line 1296: error: > > expression must be a modifiable lvalue > > clientCommand.insert(++clientCommand.begin(), > this->ServerURL.c_str()); > >^ > > > > Also these, which I got around by adding #include > > "/usr/apps/vis/ParaView/repo/ParaView/ParaViewCore/ClientServerCore/Co > re/vtkProcessModule.cxx", > line 57: error: > > identifier "LC_NUMERIC" is undefined > > setlocale(LC_NUMERIC,"C"); > > ^ > > > > "/usr/apps/vis/ParaView/repo/ParaView/ParaViewCore/ClientServerCore/Co > re/vtkProcessModule.cxx", > line 57: error: > > identifier "setlocale" is undefined > > setlocale(LC_NUMERIC,"C"); > > ^ > > > > And this, which I got around by adding #include > > > > "/usr/apps/vis/ParaView/repo/ParaView/VTK/Filters/AMR/vtkAMREnzoReader > .h", > line 70: error: > > namespace "std" has no member "string" > > int GetIndexFromArrayName( std::string arrayName ); > > > > > > Can someone suggest a fix for the lvalue error? > > Thanks, > > Mark > > > > > ___ > Powered by www.kitware.com > > Visit other Kitware open-source projects at > http://www.kitware.com/opensource/opensource.html > > Please keep messages on-topic and check the ParaView Wiki at: > http://paraview.org/Wiki/ParaView > > Follow this link to subscribe/unsubscribe: > http://www.paraview.org/mailman/listinfo/paraview > ___ Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Please keep messages on-topic and check the ParaView Wiki at: http://paraview.org/Wiki/ParaView Follow this link to subscribe/unsubscribe: http://www.paraview.org/mailman/listinfo/paraview
Re: [Paraview] paraview git build with pgi 12.4 errors
Mark, Can you try the attached patch? It should resolve the lvalue errors. Thanks Utkarsh On Fri, Aug 17, 2012 at 12:40 PM, Vanmoer, Mark W wrote: > Hello, I’m attempting to build ParaView from the git repo with PGI 12.4 and > I get the following errors for vtkSMTestDriver.cxx that I don’t know how to > fix: > > > > "/usr/apps/vis/ParaView/repo/ParaView/Utilities/TestDriver/vtkSMTestDriver.cxx", > line 434: error: > > expression must be a modifiable lvalue > > commandLine.insert(++commandLine.begin(), "--multi-servers"); > >^ > > > > "/usr/apps/vis/ParaView/repo/ParaView/Utilities/TestDriver/vtkSMTestDriver.cxx", > line 1296: error: > > expression must be a modifiable lvalue > > clientCommand.insert(++clientCommand.begin(), > this->ServerURL.c_str()); > >^ > > > > Also these, which I got around by adding #include > > "/usr/apps/vis/ParaView/repo/ParaView/ParaViewCore/ClientServerCore/Core/vtkProcessModule.cxx", > line 57: error: > > identifier "LC_NUMERIC" is undefined > > setlocale(LC_NUMERIC,"C"); > > ^ > > > > "/usr/apps/vis/ParaView/repo/ParaView/ParaViewCore/ClientServerCore/Core/vtkProcessModule.cxx", > line 57: error: > > identifier "setlocale" is undefined > > setlocale(LC_NUMERIC,"C"); > > ^ > > > > And this, which I got around by adding #include > > > > "/usr/apps/vis/ParaView/repo/ParaView/VTK/Filters/AMR/vtkAMREnzoReader.h", > line 70: error: > > namespace "std" has no member "string" > > int GetIndexFromArrayName( std::string arrayName ); > > > > > > Can someone suggest a fix for the lvalue error? > > Thanks, > > Mark > > > > > ___ > Powered by www.kitware.com > > Visit other Kitware open-source projects at > http://www.kitware.com/opensource/opensource.html > > Please keep messages on-topic and check the ParaView Wiki at: > http://paraview.org/Wiki/ParaView > > Follow this link to subscribe/unsubscribe: > http://www.paraview.org/mailman/listinfo/paraview > 0001-Fix-compiler-errors-on-pgi.patch Description: Binary data ___ Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Please keep messages on-topic and check the ParaView Wiki at: http://paraview.org/Wiki/ParaView Follow this link to subscribe/unsubscribe: http://www.paraview.org/mailman/listinfo/paraview
