[PATCH] D13802: [OPENMP] Make -fopenmp to turn on OpenMP support by default.

[Chandler Carruth via llvm-commits]

chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

Sorry I was a bit slow -- was out when commented in Nov, and was slow catching back up the past two weeks.

I've done some more testing, and the check-libomp is passing very nicely now! =D Ship it! Also make sure you follow up with the broader set of build bots to have them include this repository in the checkout.

Minor documentation tweak below, but feel free to land everything. =D I'm excited, it'll be interesting to see if anything else breaks, but so far it seems to be working well for me.


================
Comment at: docs/GettingStarted.rst:58-62
@@ -57,1 +57,7 @@
 
+#. Checkout Libomp (required for OpenMP support) **[Optional]**:
+
+   * ``cd where-you-want-llvm-to-live``
+   * ``cd llvm/projects``
+   * ``svn co http://llvm.org/svn/llvm-project/openmp/trunk openmp``
+
----------------
I'd skip the optional tag -- as you've nicely documented, its required for OpenMP support.


http://reviews.llvm.org/D13802