{"push_id":1465607406,"size":16,"distinct_size":16,"ref":"refs/heads/new_lowering","head":"e0b6da6da3c2d91599171cd82c7cd6ed3abd4779","before":"0835245a64189724166a95ab5fbbde090f88cf0c","commits":[{"sha":"3efe802b4e854017f1e120006168bcd5056254dc","author":{"name":"Devin Coughlin","email":"db071433c1a8a6fac13a68baeb91ec9cc0eae24d@apple.com"},"message":"[analyzer] Add sink after construction of temporary with no-return destructor.\n\nThe analyzer's CFG currently doesn't have nodes for calls to temporary\ndestructors. This causes the analyzer to explore infeasible paths in which\na no-return destructor would have stopped exploration and so results in false\npositives when no-return destructors are used to implement assertions.\n\nTo mitigate these false positives, this patch stops generates a sink after\nevaluating a constructor on a temporary object that has a no-return destructor.\nThis results in a loss of coverage because the time at which the destructor is\ncalled may be after the time of construction (especially for lifetime-extended\ntemporaries).\n\nThis addresses PR15599.\n\nrdar://problem/29131566\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@290140 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/wsmoses/Cilk-Clang/commits/3efe802b4e854017f1e120006168bcd5056254dc"},{"sha":"64caf82375f4a7988a0b23fa54fc527944356d3c","author":{"name":"Jordan Rose","email":"92520b00b4dfdc269f73534aec3bca632e7dd331@apple.com"},"message":"Don't try to emit nullability fix-its within/around macros.\n\nThe newly-added notes from r290132 are too noisy even when the fix-it\nis valid. For the existing warning from r286521, it's probably the\nright decision 95% of the time to put the change outside the macro if\nthe array is outside the macro and inside otherwise, but I don't want\nto overthink it right now.\n\nCaught by the ASan bot!\n\nMore rdar://problem/29524992\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@290141 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/wsmoses/Cilk-Clang/commits/64caf82375f4a7988a0b23fa54fc527944356d3c"},{"sha":"49d3871b4ea248afa47363a03b1cfb63be88682e","author":{"name":"Devin Coughlin","email":"db071433c1a8a6fac13a68baeb91ec9cc0eae24d@apple.com"},"message":"[analyzer] Add checker modeling gtest APIs.\n\ngtest is a widely-used unit-testing API. It provides macros for unit test\nassertions:\n\n  ASSERT_TRUE(p != nullptr);\n\nthat expand into an if statement that constructs an object representing\nthe result of the assertion and returns when the assertion is false:\n\n  if (AssertionResult gtest_ar_ = AssertionResult(p == nullptr))\n      ;\n  else\n    return ...;\n\nUnfortunately, the analyzer does not model the effect of the constructor\nprecisely because (1) the copy constructor implementation is missing from the\nthe header (so it can't be inlined) and (2) the boolean-argument constructor\nis constructed into a temporary (so the analyzer decides not to inline it since\nit doesn't reliably call temporary destructors right now).\n\nThis results in false positives because the analyzer does not realize that the\nthe assertion must hold along the non-return path.\n\nThis commit addresses the false positives by explicitly modeling the effects\nof the two un-inlined constructors on the AssertionResult state.\n\nI've added a new package, \"apiModeling\", for these kinds of checkers that\nmodel APIs but don't emit any diagnostics. I envision all the checkers in\nthis package always being on by default.\n\nThis addresses the false positives reported in PR30936.\n\nDifferential Revision: https://reviews.llvm.org/D27773\n\nrdar://problem/22705813\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@290143 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/wsmoses/Cilk-Clang/commits/49d3871b4ea248afa47363a03b1cfb63be88682e"},{"sha":"bfe473e6541e69accbfe25fad1af5bba82d1e76b","author":{"name":"Paul Robinson","email":"15dcd9e768f51fbcd74710dda142c061c87ab2ad@sony.com"},"message":"Make another test insensitive to the default C++ dialect.\n\nDifferential Revision: http://reviews.llvm.org/D27955\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@290145 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/wsmoses/Cilk-Clang/commits/bfe473e6541e69accbfe25fad1af5bba82d1e76b"},{"sha":"3b687020f5a628a733147c2f71ebdd7e3554e30c","author":{"name":"Richard Smith","email":"623b07eec604b9e26cffdaff9133eaf2c80f18f3@metafoo.co.uk"},"message":"Fix completely bogus types for some builtins:\n\n * In C++, never create a FunctionNoProtoType for a builtin (fixes C++1z\n   crasher from r289754).\n\n * Fix type of __sync_synchronize to be a no-parameter function rather than a\n   varargs function. This matches GCC.\n\n * Fix type of vfprintf to match its actual type. We gave it a wrong type due\n   to PR4290 (apparently autoconf generates invalid code and expects compilers\n   to choke it down or it miscompiles the program; the relevant error in clang\n   was downgraded to a warning in r122744 to fix other occurrences of this\n   autoconf brokenness, so we don't need this workaround any more).\n\n * Turn off vararg argument checking for __noop, since it's not *really* a\n   varargs function. Alternatively we could add custom type checking for it\n   and synthesize parameter types matching the actual arguments in each call,\n   but that seemed like overkill.\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@290146 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/wsmoses/Cilk-Clang/commits/3b687020f5a628a733147c2f71ebdd7e3554e30c"},{"sha":"598b67703801390463844547138d8fdee0061477","author":{"name":"George Burgess IV","email":"4d4c940392f73e96f44e4aeb7e9ed8642baafaf1@gmail.com"},"message":"Add the alloc_size attribute to clang.\n\nThis patch does three things:\n\n- Gives us the alloc_size attribute in clang, which lets us infer the\n  number of bytes handed back to us by malloc/realloc/calloc/any user\n  functions that act in a similar manner.\n- Teaches our constexpr evaluator that evaluating some `const` variables\n  is OK sometimes. This is why we have a change in\n  test/SemaCXX/constant-expression-cxx11.cpp and other seemingly\n  unrelated tests. Richard Smith okay'ed this idea some time ago in\n  person.\n- Uniques some Blocks in CodeGen, which was reviewed separately at\n  D26410. Lack of uniquing only really shows up as a problem when\n  combined with our new eagerness in the face of const.\n\nDifferential Revision: https://reviews.llvm.org/D14274\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@290149 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/wsmoses/Cilk-Clang/commits/598b67703801390463844547138d8fdee0061477"},{"sha":"ce25d00e2e923bc08d758f8fecd526d5877407ab","author":{"name":"Adrian Prantl","email":"3b76fbf1c74594027f839a53c333f1ac0d409492@apple.com"},"message":"Update for LLVM global variable debug info API change.\n\nThis reapplies r289921.\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@290155 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/wsmoses/Cilk-Clang/commits/ce25d00e2e923bc08d758f8fecd526d5877407ab"},{"sha":"0d618872cdc401a702ca00285e409281753dcd9b","author":{"name":"Akira Hatanaka","email":"9b0cf1190e34e4dab0e0e0463e917e57d98208cb@apple.com"},"message":"[Parser] Correct typo after lambda capture initializer is parsed.\n\nThis patch fixes an assertion that is triggered when RecordLayoutBuilder\ntries to compute the size of a field (for capture \"name\" in the test\ncase) whose type hasn't been deduced. The patch fixes the bug by\ncorrecting the typo of the capture initializer after the initializer is\nparsed and before setting the expression for the annotation token.\n\nFixes PR30566.\n\nrdar://problem/23380132\n\nDifferential Revision: https://reviews.llvm.org/D25206\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@290156 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/wsmoses/Cilk-Clang/commits/0d618872cdc401a702ca00285e409281753dcd9b"},{"sha":"3a4d285c59799871e39d1d37580be9f19e24f4f0","author":{"name":"Chandler Carruth","email":"f202c0deeaccee703d9195ab786f78f6938b11a3@gmail.com"},"message":"Fix the spelling of 'bitfield' in diagnostics to be consistently 'bit-field'.\n\nThe latter agrees with most existing diagnostics and the C and C++ standards.\n\nDifferential Revision: https://reviews.llvm.org/D26530\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@290159 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/wsmoses/Cilk-Clang/commits/3a4d285c59799871e39d1d37580be9f19e24f4f0"},{"sha":"4e23c3c7878efe0251fcf3af27231635a438359e","author":{"name":"Richard Smith","email":"623b07eec604b9e26cffdaff9133eaf2c80f18f3@metafoo.co.uk"},"message":"Add test for DR692.\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@290166 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/wsmoses/Cilk-Clang/commits/4e23c3c7878efe0251fcf3af27231635a438359e"},{"sha":"4e57f52faea670d545cbe937f52e6e9cf5d19b0b","author":{"name":"Chandler Carruth","email":"f202c0deeaccee703d9195ab786f78f6938b11a3@gmail.com"},"message":"Revert r290149: Add the alloc_size attribute to clang.\n\nThis commit fails MSan when running test/CodeGen/object-size.c in\na confusing way. After some discussion with George, it isn't really\nclear what is going on here. We can make the MSan failure go away by\ntesting for the invalid bit, but *why* things are invalid isn't clear.\nAnd yet, other code in the surrounding area is doing precisely this and\ntesting for invalid.\n\nGeorge is going to take a closer look at this to better understand the\nnature of the failure and recommit it, for now backing it out to clean\nup MSan builds.\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@290169 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/wsmoses/Cilk-Clang/commits/4e57f52faea670d545cbe937f52e6e9cf5d19b0b"},{"sha":"b49f706eef3ac5cdadd86aaac5a50335cf8d895e","author":{"name":"Egor Churaev","email":"d3102030955bb7d50887cbea23a1814bc5ff856d@gmail.com"},"message":"[OpenCL] Enabling the usage of CLK_NULL_QUEUE as compare operand.\n\nSummary: Enabling the compression of CLK_NULL_QUEUE to variable of type queue_t.\n\nReviewers: Anastasia\n\nSubscribers: cfe-commits, yaxunl, bader\n\nDifferential Revision: https://reviews.llvm.org/D27569\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@290171 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/wsmoses/Cilk-Clang/commits/b49f706eef3ac5cdadd86aaac5a50335cf8d895e"},{"sha":"add07b73fd0f5cd4110bf6c944112df9350380d5","author":{"name":"Kevin Funk","email":"d4a9ad893a3b3d9bd60ca464f3945024db54b0de@kde.org"},"message":"Fix for clang_Cursor_getSpellingNameRange()\n\nSummary:\nFixes spelling name ranges for user-defined string literal operators.\n\nExample:\n  constexpr int operator\"\"_toint(unsigned long long val)\n  { return int(val); }\n\nBefore this patch the spelling name range on consisted of 'operator'.\n\nAfter this patch: 'operator\"\"_toint'.\n\nRelated to http://reviews.llvm.org/D5041, which fixes the function for\nother cursor kinds.\n\nReviewers: akyrtzi, craigt, skalinichev, klimek, milianw, bkramer\n\nSubscribers: cfe-commits\n\nTags: #clang-c\n\nDifferential Revision: https://reviews.llvm.org/D18462\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@290172 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/wsmoses/Cilk-Clang/commits/add07b73fd0f5cd4110bf6c944112df9350380d5"},{"sha":"04092174f77a5cba9a2ff50d88002ffd82e478a5","author":{"name":"Daniel Jasper","email":"3b340437527f53faa3c9db5bbc30419c09b4e1fa@google.com"},"message":"Revert \"[OpenCL] Enabling the usage of CLK_NULL_QUEUE as compare operand.\"\n\nThis reverts commit r290171. It triggers a bunch of warnings, because\nthe new enumerator isn't handled in all switches. We want a warning-free\nbuild.\n\nReplied on the commit with more details.\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@290173 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/wsmoses/Cilk-Clang/commits/04092174f77a5cba9a2ff50d88002ffd82e478a5"},{"sha":"49d944aa259dae5ee57ee43f76e455361a7d33fb","author":{"name":"Alexey Bataev","email":"7e1552918a374ea04d5b75d8d91309d1499b18ee@hotmail.com"},"message":"[OPENMP] Fix for PR31428: variable named like directive name modifier\n\nDirective name modifiers in 'if' clause are allowed only for OpenMP 4.5\nand higher + in OpenMP 4.5 parsing procedure emits error message if ':'\nis not found after directive name modifier.\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@290175 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/wsmoses/Cilk-Clang/commits/49d944aa259dae5ee57ee43f76e455361a7d33fb"},{"sha":"e0b6da6da3c2d91599171cd82c7cd6ed3abd4779","author":{"name":"TB Schardl","email":"fba17f1e273f74524da24918003de6208f9d4957@mit.edu"},"message":"Merge branch 'master' of github.com:llvm-mirror/clang into new_lowering","distinct":true,"url":"https://api.github.com/repos/wsmoses/Cilk-Clang/commits/e0b6da6da3c2d91599171cd82c7cd6ed3abd4779"}]}
{"actor":{"display_login":"neboat"}}