Hi Eli and Dmitri,

Let me start again from the beginning one step at a time. Going
further, I avoid sending more than one patch at a time as reviewing
(big) several patches requires considerable effort, and also some time
it is confusing. Attached is the very first patch to support -fopenmp
option. This patch is in fact smaller one, and it already went through
few rounds of review, and all the review comments are taken for it.

However, please take one more look at it, and if it is in a good shape
to commit, please do the same or provide me an approval to commit this
patch. OTOH, if there is any fresh review comments, I will work on it.

Note: Chandler as an opinion to support counter part of this option,
namely -fno-openmp. I am still thinking, if it is worth supporting it
as I am not aware of any other OpenMP compiler supporting such an
option. However, let us think about it, and if we decide that it is
useful to support it, I will support it.


-- 
mahesha

Attachment: fopenmp_option_support.patch
Description: Binary data

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to