On Thu, May 28, 2015 at 5:09 AM İsmail Dönmez <ism...@donmez.ws> wrote:
> Hi, > > On Thu, May 28, 2015 at 4:52 AM, Chandler Carruth <chandl...@gmail.com> > wrote: > > Author: chandlerc > > Date: Wed May 27 20:52:38 2015 > > New Revision: 238389 > > > > URL: http://llvm.org/viewvc/llvm-project?rev=238389&view=rev > > Log: > > [omp] Re-work Clang's handling of -fopenmp and undo r237769. > > > This seems to break openmp tests with > -DCLANG_DEFAULT_OPENMP_RUNTIME=libiomp5 see: > Yea, I wanted to have test coverage of the default, but it doesn't work well when the default is overridden. I looked into how to actually have test coverage of the default and track which default was selected, but its really hard and complex. Yuck. I'll look for a way to get at least *some* coverage of the default without making it break folks that are overriding it. Sorry for the trouble.
_______________________________________________ cfe-commits mailing list cfe-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
