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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to