[cfe-commits] [PATCH] First OpenMP patch

[Chandler Carruth]

On Mon, Dec 17, 2012 at 9:14 AM, Dmitri Gribenko <gribozavr at gmail.com>wrote:

> What is more important than documentation, is the change in behavior.
> Someone might have used '-fopenmp' with Clang before, and now they
> will get an incomplete implementation.  We might also introduce a
> warning to Clang like "experimental OpenMP support enabled" for the
> time being.


Just to be clear, this is what I was worried about.

Previously, the '-fopenmp' flag was handled just like many GCC flags that
LLVM will accept and ignore. Now there appears to be support for the
feature, but it doesn't do what the user wants...

Anyways, the real point is that we should have some write up (preferably
committed) which describes the plan and rationale for OpenMP support in
Clang. Otherwise we're likely te stay in the exact spot you point out Hal:
FIXME's all over the place for unimplemented functionality. =/ That's
actually what I'm hoping to avoid.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121217/3a4fd44d/attachment.html>