[cfe-commits] [LLVMdev] [cfe-dev] OpenMP support in CLANG: A proposal

[Eli Friedman]

On Tue, Oct 23, 2012 at 3:26 AM, Mahesha HS <mahesha.llvm at gmail.com> wrote:
> Hi Eli,
>
> Attached (openmp-enum-data-structure-support.patch) is the patch no 3,
> which implements the basic data structures required for OpenMP
> parsing.  At this point, I could not attach any relevant test-case(s)
> for this patch. However, when I submit the next patch which will be
> related OpenMP parsing, I should be able to attach with it, the
> relevant test-case(s) for the current patch.
>
> Is the (revised) patch no 2 (omp_pragma_registration_revised.patch),
> which was sent along with my previous mail fine? Also, what about the
> checking-in patch no 1 (fopenmp_option_support.patch) to trunk?

Sorry about the delay; I replied to the previous email.

+/// \brief DefaultMap - A StringMap which holds enum kinds associated
+/// with different OpenMP *default clause kind* names.
+DefaultOwningPtr DefaultMap;

Global variables aren't allowed in clang libraries.  Probably the
simplest thing to do would be to switch to a const table and use a
binary search.

-Eli