[Openmp-commits] [PATCH] D35719: [CMake] Disable building libomptarget and add CMake switch
Jonas Hahnfeld via Phabricator via Openmp-commits
openmp-commits at lists.llvm.org
Sun Jul 23 23:19:03 PDT 2017
Hahnfeld marked 3 inline comments as done.
Hahnfeld added inline comments.
Comment at: CMakeLists.txt:8
# Currently libomptarget cannot be compiled on Windows or MacOS X.
> Hahnfeld wrote:
> > hans wrote:
> > > Since the tests fail by default, I think it should be disabled in the build by default.
> > >
> > > trunk should always be in a working state.
> > Well, it's broken since January so nobody cared enough. @gtbercea, what do you think?
> Most people don't check out or build the openmp repo, and your buildbot doesn't run the tests -- which is another reason the code shouldn't be built by default.
Updated the patch. I'll let it sit for two days or so to give the others time to respond.
More information about the Openmp-commits