{"push_id":3182619826,"size":6,"distinct_size":6,"ref":"refs/heads/master","head":"0ee7e92f1d104e37043b621fd50aec4cd48e5fb5","before":"66e51a0bb9bbe39b1aee3cedfdeb8ad39688cafd","commits":[{"sha":"f21e62190820e0ae3570ba617043e42a64c1582b","author":{"name":"Joel E. Denny","email":"48cfda3b1ab5e74154db2af297f40ae4bd0380fe@gmail.com"},"message":"[OpenMP] Replace predetermined shared for const variable\n\nThe following appears in OpenMP 3.1 sec. 2.9.1.1 as a predetermined\ndata-sharing attribute:\n\n> Variables with const-qualified type having no mutable member are\n> shared.\n\nIt does not appear in OpenmP 4.0, 4.5, or 5.0.  This patch removes the\nimplementation of that attribute when the requested OpenMP version is\ngreater than 3.1.\n\nOne effect of that removal is that `default(none)` affects const\nvariables without mutable members.\n\nAlso, without this patch, if a const variable without mutable members\nwas explicitly lastprivate or private, it was an error because it was\npredetermined shared.  Now, clang instead complains that it's const\nwithout mutable fields, which is a more intelligible diagnostic.  That\nshould be fine for all of the above versions because they all have\nsomething like the following, which is quoted from OpenMP 5.0\nsec. 2.19.3:\n\n> A variable that is privatized must not have a const-qualified type\n> unless it is of class type with a mutable member. This restriction does\n> not apply to the firstprivate clause.\n\nreduction and linear clauses already have separate checks for const\nvariables.  Future patches will merge the implementations.\n\nReviewed By: ABataev\n\nDifferential Revision: https://reviews.llvm.org/D56113\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@350439 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/bogner/llvm-zipper-prototype/commits/f21e62190820e0ae3570ba617043e42a64c1582b"},{"sha":"751ff6101c4f969a43dc15b36defb2787ef426b2","author":{"name":"LLVM Monorepo conversion","email":"cd21cd76f101be56d104584fc5a716fe3b563fb7@llvm.org"},"message":"Merging r350439: \"[OpenMP] Replace predetermined shared for const variable\"\n\ngit-crossrepo-prefix: clang/","distinct":true,"url":"https://api.github.com/repos/bogner/llvm-zipper-prototype/commits/751ff6101c4f969a43dc15b36defb2787ef426b2"},{"sha":"5b0be04e490e2b66c2abb51292e01112b6834ef4","author":{"name":"Joel E. Denny","email":"48cfda3b1ab5e74154db2af297f40ae4bd0380fe@gmail.com"},"message":"[OpenMP] Refactor const restriction for reductions\n\nAs discussed in D56113, this patch refactors the implementation of the\nconst restriction for reductions to reuse a function introduced by\nD56113.  A side effect is that diagnostics sometimes now say\n\"variable\" instead of \"list item\" when a list item is a variable.\n\nReviewed By: ABataev\n\nDifferential Revision: https://reviews.llvm.org/D56298\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@350440 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/bogner/llvm-zipper-prototype/commits/5b0be04e490e2b66c2abb51292e01112b6834ef4"},{"sha":"77cfaa3cdaf718af03c5cf51be7d6b40687e35ab","author":{"name":"LLVM Monorepo conversion","email":"cd21cd76f101be56d104584fc5a716fe3b563fb7@llvm.org"},"message":"Merging r350440: \"[OpenMP] Refactor const restriction for reductions\"\n\ngit-crossrepo-prefix: clang/","distinct":true,"url":"https://api.github.com/repos/bogner/llvm-zipper-prototype/commits/77cfaa3cdaf718af03c5cf51be7d6b40687e35ab"},{"sha":"3bd81ff22212f694b92736739bfcaccd86da737c","author":{"name":"Joel E. Denny","email":"48cfda3b1ab5e74154db2af297f40ae4bd0380fe@gmail.com"},"message":"[OpenMP] Refactor const restriction for linear\n\nAs discussed in D56113, this patch refactors the implementation of the\nconst restriction for linear to reuse a function introduced by D56113.\nA side effect is that, if a variable has mutable members, this\ndiagnostic is now skipped, and the diagnostic for the variable not\nbeing an integer or pointer is reported instead.\n\nReviewed By: ABataev\n\nDifferential Revision: https://reviews.llvm.org/D56299\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@350441 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/bogner/llvm-zipper-prototype/commits/3bd81ff22212f694b92736739bfcaccd86da737c"},{"sha":"0ee7e92f1d104e37043b621fd50aec4cd48e5fb5","author":{"name":"LLVM Monorepo conversion","email":"cd21cd76f101be56d104584fc5a716fe3b563fb7@llvm.org"},"message":"Merging r350441: \"[OpenMP] Refactor const restriction for linear\"\n\ngit-crossrepo-prefix: clang/","distinct":true,"url":"https://api.github.com/repos/bogner/llvm-zipper-prototype/commits/0ee7e92f1d104e37043b621fd50aec4cd48e5fb5"}]}
{"actor":{"display_login":"bogner"}}