[PATCH] D65272: Update some tests after r366702

2019-07-25 Thread Krasimir Georgiev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL367012: Update some tests after r366702 (authored by krasimir, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Repository: rL LLVM CHANGES SINCE LAST ACTION htt

[PATCH] D65272: Update some tests after r366702

2019-07-25 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. This looks safe and consistent with the old tests and intent. Approving to unblock CI Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65272/new/ https://r

[PATCH] D65272: Update some tests after r366702

2019-07-25 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir created this revision. Herald added a reviewer: jdoerfert. Herald added a project: clang. Herald added a subscriber: cfe-commits. r366702 added a set of new clang-cl -- specific openmp flags together with tests. The way the newly added tests work is problematic: consider for example this