[PATCH] D71884: [OpenMP] Fix formatting of OpenMP error message.

2020-01-02 Thread Alexey Bataev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG87a004d0f8c2: [OpenMP] Fix formatting of OpenMP error message, by Wang Tianqing. (authored by ABataev). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71884/n

[PATCH] D71884: [OpenMP] Fix formatting of OpenMP error message.

2020-01-01 Thread Wang Tianqing via Phabricator via cfe-commits
tianqing added a comment. Could someone help commit this patch? Thanks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71884/new/ https://reviews.llvm.org/D71884 ___ cfe-commits mailing list cfe-commits

[PATCH] D71884: [OpenMP] Fix formatting of OpenMP error message.

2019-12-25 Thread Johannes Doerfert via Phabricator via cfe-commits
jdoerfert accepted this revision. jdoerfert added a comment. This revision is now accepted and ready to land. I would have just stripped the last chars of ` Out.str()` but if this passes all the tests I'm fine with it. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION ht

[PATCH] D71884: [OpenMP] Fix formatting of OpenMP error message.

2019-12-25 Thread Wang Tianqing via Phabricator via cfe-commits
tianqing created this revision. Herald added subscribers: cfe-commits, guansong. Herald added a reviewer: jdoerfert. Herald added a project: clang. tianqing added a reviewer: ABataev. `getListOfPossibleValues()` formatted incorrectly when there is only one value, emitting something like `expected