Hi, You should make test/Driver/openmp-options.c use FileCheck, rather than grep.
+ // If -fopenmp enablement fails, then check, if -fno-openmp is need to be + // enabled. This comment confuses me, when does this happen? Also, there doesn't seem to be a test for this case. Otherwise, looks good. Thanks, Joey -----Original Message----- From: [email protected] [mailto:[email protected]] On Behalf Of Mahesha HS Sent: 21 November 2012 14:36 To: llvm cfe Subject: [cfe-commits] [PATCH] First OpenMP patch Hi all, Here is the first OpenMP patch. It supports options -fopenmp and -fno-openmp. When only -fopenmp is passed, OpenMP is enabled. When only -fno-openmp is passed, OpenMP is disabled. When both the options are passed, the one which appear later in the command line wins. I am re-sending this patch third time. I am positively hoping that - this time - this patch will get checked-in into trunk :) -- mahesha _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
