From: Sean M. <se...@ro...> - 2010-07-08 16:08:38
|
Hi all, I'm new to this list, and hope this post is on-topic. I don't actually use vxl, but I do use ITK, which in turn uses vxl. I am attempting to build *my* code with the clang compiler but it is currently failing due to syntax errors in vxl (by way of me including ITK headers). So I thought the first step would be to create a vxl dashboard using clang. Naturally, there are syntax errors. But it seems there is a bigger problem: vxl doesn't even build with gcc. The only Mac dashboard has been failing for days: <http://www.cdash.org/CDash/viewBuildError.php?buildid=659123> Is this a known issue? Thanks, -- ____________________________________________________________ Sean McBride, B. Eng se...@ro... Rogue Research www.rogue-research.com Mac Software Developer Montréal, Québec, Canada |
From: Gehua Y. <yan...@gm...> - 2010-07-08 18:48:59
|
Sean, I have VXL compiled on my mac about one month ago. But I have turned "BUILD_BRL" and a few other university contributions off in the CMake settings. Gary Yang DualAlign LLC On Jul 8, 2010, at 12:08 PM, Sean McBride wrote: > Hi all, > > I'm new to this list, and hope this post is on-topic. > > I don't actually use vxl, but I do use ITK, which in turn uses vxl. > > I am attempting to build *my* code with the clang compiler but it is > currently failing due to syntax errors in vxl (by way of me including > ITK headers). > > So I thought the first step would be to create a vxl dashboard using > clang. Naturally, there are syntax errors. But it seems there is a > bigger problem: vxl doesn't even build with gcc. The only Mac dashboard > has been failing for days: > > <http://www.cdash.org/CDash/viewBuildError.php?buildid=659123> > > Is this a known issue? > > Thanks, > > -- > ____________________________________________________________ > Sean McBride, B. Eng se...@ro... > Rogue Research www.rogue-research.com > Mac Software Developer Montréal, Québec, Canada > > > > ------------------------------------------------------------------------------ > This SF.net email is sponsored by Sprint > What will you do first with EVO, the first 4G phone? > Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first > _______________________________________________ > Vxl-users mailing list > Vxl...@li... > https://lists.sourceforge.net/lists/listinfo/vxl-users |
From: Sean M. <se...@ro...> - 2010-07-08 20:16:28
|
On Thu, 8 Jul 2010 15:04:30 -0400, Isabel Restrepo said: >I host the Mac OS X build for the dashboard. The dashboard errors are >due to opencl code in a brl contrib area, otherwise vxl builds fine in >Mac OS (I think VTK doesn't use vxl contrib libraries). There are many >other linux platforms that build vxl successfully so building with gcc >is not a problem. Strange that Mac OS, of all platforms, would have an OpenCL build problem. :) I've tried again with BUILD_BRL off and now there are only 2 errors when building with gcc. See: <http://www.cdash.org/CDash/viewBuildError.php?buildid=659439> Do you know what causes them? And if they are a known problem, what setting can I turn off? Once my gcc dashboard works, I can try again with clang. Cheers, -- ____________________________________________________________ Sean McBride, B. Eng se...@ro... Rogue Research www.rogue-research.com Mac Software Developer Montréal, Québec, Canada |
From: Isabel R. <mi...@le...> - 2010-07-08 20:24:45
|
Tthe thing is none of the other builds are building with OpenCL, and the people using OpenCL use windows SDKs... It seems the other build errors are in MUL which is another contrib library so try turning it off in CMake Isabel On Jul 8, 2010, at 4:16 PM, Sean McBride wrote: > On Thu, 8 Jul 2010 15:04:30 -0400, Isabel Restrepo said: > >> I host the Mac OS X build for the dashboard. The dashboard errors are >> due to opencl code in a brl contrib area, otherwise vxl builds fine in >> Mac OS (I think VTK doesn't use vxl contrib libraries). There are many >> other linux platforms that build vxl successfully so building with gcc >> is not a problem. > > Strange that Mac OS, of all platforms, would have an OpenCL build problem. :) > > I've tried again with BUILD_BRL off and now there are only 2 errors when > building with gcc. See: > <http://www.cdash.org/CDash/viewBuildError.php?buildid=659439> > > Do you know what causes them? And if they are a known problem, what > setting can I turn off? > > Once my gcc dashboard works, I can try again with clang. > > Cheers, > > -- > ____________________________________________________________ > Sean McBride, B. Eng se...@ro... > Rogue Research www.rogue-research.com > Mac Software Developer Montréal, Québec, Canada > > ---------------------------------------- Maria Isabel Restrepo Ph.D Candidate Computer Vision Brown University mar...@br... |
From: Sean M. <se...@ro...> - 2010-07-09 14:07:00
|
On Thu, 8 Jul 2010 16:24:29 -0400, Isabel Restrepo said: >It seems the other build errors are in MUL which is another contrib >library so try turning it off in CMake That worked, thanks! Now it builds with gcc: <http://www.cdash.org/CDash/buildSummary.php?buildid=660210> Then I tried another build with all the same settings except using clang and there are hundreds of errors: <http://www.cdash.org/CDash/viewBuildError.php?buildid=660226> I'm not a C++ expert, but many of the errors look legitimate. Anyone interested in helping getting this building? As I said, I've used vxl, but I can spare a little time working on the dashboard, filing bugs with the clang folks, etc. Cheers, -- ____________________________________________________________ Sean McBride, B. Eng se...@ro... Rogue Research www.rogue-research.com Mac Software Developer Montréal, Québec, Canada |
From: Peter V. <pet...@ya...> - 2010-07-29 20:21:14
|
On 2010-07-09, Sean McBride <se...@ro...> wrote: > [...] > Then I tried another build with all the same settings > except using clang and there are hundreds of errors: > <http://www.cdash.org/CDash/viewBuildError.php?buildid=660226> > > I'm not a C++ expert, but many of the errors look legitimate. > > Anyone interested in helping getting this building? I've looked at these errors, and it seems like most of them happen in two cases: the first is when a method of a templated parent class is called (from a method of a derived class). This is easily fixed by prefixing that call with the parent class name and "::" which I've actually done in a few cases (and SVN committed). (In those cases, with the added prefix the code even becomes more readable since it's now clear where the method belongs to.) The second case, and actually the cause of most errors from the clang compiler in vxl, is when a vnl_matrix_fixed<T,n,m> is used where a vnl_matrix<T> and is expected, and similarly for vnl_vector<T> & vnl_vector_fixed<T,n>. Although there is the following convertor: operator const vnl_vector_ref<T>() const in class vnl_vector_fixed, and vnl_vector_ref is a derived class from vnl_vector, the clang compiler does not automatically use this to allow a vnl_vector_fixed in contexts where a vnl_vector is expected. Again, I've modified some of these cases, thereby also cleaning up a bit, in those cases where mixing vnl_vector and vnl_vector_fixed was not a good idea. In most cases, I've inserted an explicit vnl_vector (or vnl_matrix) constructor, which costs a memcopy; in most cases these should be replaced by using the as_ref() method of vnl_vector_fixed to avoid the memcopy; I'll do that one of these days. From yesterday's dashboard of the clang compiler, it seems like these modifications will help to reduce the number of errors. (And the other builds look very green today -- seems nothing got broken.) To be continued... Any feedback is of course welcome! And please keep us informed about your clang+vxl experiences. -- Peter. |
From: Sean M. <se...@ro...> - 2010-07-30 19:51:28
|
On Thu, 29 Jul 2010 20:21:06 +0000, Peter Vanroose said: >> Then I tried another build with all the same settings >> except using clang and there are hundreds of errors: >> <http://www.cdash.org/CDash/viewBuildError.php?buildid=660226> >> >> I'm not a C++ expert, but many of the errors look legitimate. >> >> Anyone interested in helping getting this building? > >I've looked at these errors, and > > *SNIP* > >From yesterday's dashboard of the clang compiler, it seems like these >modifications will help to reduce the number of errors. (And the other >builds look very green today -- seems nothing got broken.) >To be continued... Peter, Thanks a lot for looking at this and fixing some issues! It's indeed much better now, instead of reaching the dashboard max of 500, there are now "only" 354 errors: <http://www.cdash.org/CDash/viewBuildError.php?buildid=680234> That may also be due to the fact that I updated my build of clang. They are fast improving their C++ support. One error is repeated often and looks simple enough to fix: /.../vxl/contrib/prip/vmap/vmap_ptr_sequences.h:295:5: error: use of undeclared identifier 'set_begin' set_begin(new pointer[arg_size]) ; ^ this-> It appears to be saying "this->" must be added. Likely some of the errors are bugs in clang. Just this week several of the VTK developers got VTK building with clang... There were two issues that were clang bugs, and the clang team fixed them each within a day, and now VTK builds! Cheers, -- ____________________________________________________________ Sean McBride, B. Eng se...@ro... Rogue Research www.rogue-research.com Mac Software Developer Montréal, Québec, Canada |
From: Peter V. <pet...@ya...> - 2010-07-30 21:58:58
|
Sean McBride <se...@ro...> wrote: > Thanks a lot for looking at this and fixing some issues! > It's indeed much better now, instead of reaching the > dashboard max of 500, there are now "only" 354 errors Well, of course, I didn't add all necessary ".as_ref()"s, but you can figure out for yourself (based on the compiler errors) where the other ones are needed. > One error is repeated often and looks simple enough to fix: > > /.../vxl/contrib/prip/vmap/vmap_ptr_sequences.h:295:5: > error: use of > undeclared identifier 'set_begin' > set_begin(new pointer[arg_size]) > ; > ^ > this-> > > It appears to be saying "this->" must be added. OK, did that (as well as some similar ones in that file). Most likely, contrib/prip/vmap will "never" compile with the clang compiler since it's using some "advanced" C++ stuff, so don't put too much effort into fixing that library's build. -- Peter. |
From: Sean M. <se...@ro...> - 2010-08-02 20:49:54
|
On Fri, 30 Jul 2010 21:58:50 +0000, Peter Vanroose said: >Most likely, contrib/prip/vmap will "never" compile with the clang >compiler since it's using some "advanced" C++ stuff, so don't put too >much effort into fixing that library's build. In that case, I've set BUILD_CONTRIB and BUILD_PRIP to off and now there's just a small number of errors: <http://www.cdash.org/CDash/viewBuildError.php?buildid=682893> BTW, what is vxl's minimum requirement? C++98? C++03? C++0x? clang currently claims full support of C++03 (except 'export'): <http://clang.llvm.org/cxx_status.html> Cheers, -- ____________________________________________________________ Sean McBride, B. Eng se...@ro... Rogue Research www.rogue-research.com Mac Software Developer Montréal, Québec, Canada |
From: Isabel R. <mi...@le...> - 2010-07-09 00:04:25
|
Dea Sean, I host the Mac OS X build for the dashboard. The dashboard errors are due to opencl code in a brl contrib area, otherwise vxl builds fine in Mac OS (I think VTK doesn't use vxl contrib libraries). There are many other linux platforms that build vxl successfully so building with gcc is not a problem. Isabel On Jul 8, 2010, at 12:08 PM, Sean McBride wrote: > Hi all, > > I'm new to this list, and hope this post is on-topic. > > I don't actually use vxl, but I do use ITK, which in turn uses vxl. > > I am attempting to build *my* code with the clang compiler but it is > currently failing due to syntax errors in vxl (by way of me including > ITK headers). > > So I thought the first step would be to create a vxl dashboard using > clang. Naturally, there are syntax errors. But it seems there is a > bigger problem: vxl doesn't even build with gcc. The only Mac dashboard > has been failing for days: > > <http://www.cdash.org/CDash/viewBuildError.php?buildid=659123> > > Is this a known issue? > > Thanks, > > -- > ____________________________________________________________ > Sean McBride, B. Eng se...@ro... > Rogue Research www.rogue-research.com > Mac Software Developer Montréal, Québec, Canada > > ---------------------------------------- Maria Isabel Restrepo Ph.D Candidate Computer Vision Brown University mar...@br... |