{"push_id":3010361606,"size":23,"distinct_size":23,"ref":"refs/heads/master","head":"4ddbc316e2f505a52102642c899015af41acc04f","before":"94f13e37e55a5f155cdbe4dbc7bb2cb3a5b43860","commits":[{"sha":"f9567535a365e87748140b41fa9b0f9befbf5122","author":{"name":"Hans Wennborg","email":"8800578b51f022c8d8adb9606a8b3db4fedbdac6@hanshq.net"},"message":"[clang-cl] Inherit dllexport to static locals also in template instantiations (PR39496)\n\nIn the course of D51340, @takuto.ikuta discovered that Clang fails to put\ndllexport/import attributes on static locals during template instantiation.\n\nFor regular functions, this happens in Sema::FinalizeDeclaration(), however for\ntemplate instantiations we need to do something in or around\nTemplateDeclInstantiator::VisitVarDecl(). This patch does that, and extracts\nthe code to a utility function.\n\nDifferential Revision: https://reviews.llvm.org/D53870\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345699 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/f9567535a365e87748140b41fa9b0f9befbf5122"},{"sha":"5bd4b03535c115d46ecd089fdf391efd794e67e1","author":{"name":"Hans Wennborg","email":"8800578b51f022c8d8adb9606a8b3db4fedbdac6@hanshq.net"},"message":"Follow-up to r345699: Call CheckStaticLocalForDllExport later for templates\n\nCalling it too early might cause dllimport to get inherited onto the\nVarDecl before the initializer got attached. See the test case for an\nexample where this broke things.\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345709 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/5bd4b03535c115d46ecd089fdf391efd794e67e1"},{"sha":"537ae129b767ac40785b17328ba1aaca7e5f5ace","author":{"name":"Kristof Umann","email":"110393adbfb0fd6dcfb6f9680d770d661e28ceb7@gmail.com"},"message":"[analyzer][PlistMacroExpansion] Part 1.: New expand-macros flag\n\nThis is the first part of the implementation of the inclusion of macro\nexpansions into the plist output. It adds a new flag that adds a new\n\"macro_expansions\" entry to each report that has PathDiagnosticPieces that were\nexpanded from a macro. While there's an entry for each macro expansion, both\nthe name of the macro and what it expands to is missing, and will be implemented\nin followup patches.\n\nDifferential Revision: https://reviews.llvm.org/D52742\n\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345724 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/537ae129b767ac40785b17328ba1aaca7e5f5ace"},{"sha":"e8be1bee43603c8a42950c8f855c8891b2167eb8","author":{"name":"Jonas Toth","email":"fd39b54e6c4928a0d43227b414f689f784d31d01@gmail.com"},"message":"[clang] try-fix broken documentation builder\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345737 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/e8be1bee43603c8a42950c8f855c8891b2167eb8"},{"sha":"20c21b7b6835ec96fbe9810c3e51b4777818d616","author":{"name":"Kristof Umann","email":"110393adbfb0fd6dcfb6f9680d770d661e28ceb7@gmail.com"},"message":"[Lex] Make MacroDirective::findDirectiveAtLoc take const SourceManager\n\nI'm currently working on including macro expansions in the Static Analyzer's\nplist output, where I can only access a const SourceManager.\n\nDifferential Revision: https://reviews.llvm.org/D53940\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345741 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/20c21b7b6835ec96fbe9810c3e51b4777818d616"},{"sha":"597ac06e9d4ee7609bd9b70640774845b11b4130","author":{"name":"George Karpenkov","email":"0ec62ea6484a624425cd34ea6ee504b1d60cca31@apple.com"},"message":"[analyzer] Enable retain count checking for OSObject by defa\n\nThe FP rate seems to be good enough now.\n\nDifferential Revision: https://reviews.llvm.org/D53849\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345745 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/597ac06e9d4ee7609bd9b70640774845b11b4130"},{"sha":"a156e18c50ca1c624c01414da6b653862c1bcb23","author":{"name":"George Karpenkov","email":"0ec62ea6484a624425cd34ea6ee504b1d60cca31@apple.com"},"message":"[analyzer] RetainCountChecker: for now, do not trust the summaries of inlined code\n\nTrusting summaries of inlined code would require a more thorough work,\nas the current approach was causing too many false positives, as the new\nexample in test.  The culprit lies in the fact that we currently escape\nall variables written into a field (but not passed off to unknown\nfunctions!), which can result in inconsistent behavior.\n\nrdar://45655344\n\nDifferential Revision: https://reviews.llvm.org/D53902\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345746 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/a156e18c50ca1c624c01414da6b653862c1bcb23"},{"sha":"abe726a8c420d79d8904b9f90d35f0516c7c5d4f","author":{"name":"George Karpenkov","email":"0ec62ea6484a624425cd34ea6ee504b1d60cca31@apple.com"},"message":"[analyzer] Re-add custom OSIterator rule for RetainCountChecker\n\nTurns out the rule is quite ubiquitous.\n\nRevert of https://reviews.llvm.org/D53628\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345747 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/abe726a8c420d79d8904b9f90d35f0516c7c5d4f"},{"sha":"5eb3d750c68ec9b2bef977d9bfa9e895d1d869c8","author":{"name":"Reid Kleckner","email":"ab3c2931970baa67c6d76cdeb719744d873e09ef@google.com"},"message":"Re-land r345676 \"[Win64] Handle passing i128 by value\"\n\nFix the unintended switch/case fallthrough to avoid changing long double\nbehavior.\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345748 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/5eb3d750c68ec9b2bef977d9bfa9e895d1d869c8"},{"sha":"262f33340b2a1f9d14ce812bf568bbd42ea11f23","author":{"name":"Krasimir Georgiev","email":"6900256643404ba7670b0457a96b9790d193bef4@google.com"},"message":"[clang-format] tweaked another case of lambda formatting\n\nSummary:\nThis is done in order to improve cases where the lambda's body is moved too far to the right. Consider the following snippet with column limit set to 79:\n\n```\nvoid f() {\n  leader::MakeThisCallHere(&leader_service_,\n                           cq_.get(),\n                           [this, liveness](const leader::ReadRecordReq& req,\n                                            std::function<void()> done) {\n                             logger_->HandleReadRecord(\n                                 req, resp, std::move(done));\n                           });\n\n  leader::MakeAnother(&leader_service_,\n                      cq_.get(),\n                      [this, liveness](const leader::ReadRecordReq& req,\n                                       std::function<void()> done) {\n                        logger_->HandleReadRecord(\n                            req, resp, std::move(done), a);\n                      });\n}\n```\n\nThe tool favors extra indentation for the lambda body and so the code incurs extra wrapping and adjacent calls are indented to a different level. I find this behavior annoying and I'd like the tool to favor new lines and, thus, use the extra width.\n\nThe fix, reduced, brings the following formatting.\n\nBefore:\n\n    function(1,\n             [] {\n               DoStuff();\n               //\n             },\n             1);\n\nAfter:\n\n    function(\n        1,\n        [] {\n          DoStuff();\n          //\n        },\n        1);\n\nRefer to the new tests in FormatTest.cpp\n\nContributed by oleg.smolsky!\n\nReviewers: djasper, klimek, krasimir\n\nSubscribers: cfe-commits, owenpan\n\nTags: #clang\n\nDifferential Revision: https://reviews.llvm.org/D52676\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345753 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/262f33340b2a1f9d14ce812bf568bbd42ea11f23"},{"sha":"3c95e03870283af2773cb8050040c342af18708b","author":{"name":"Gabor Marton","email":"611402bc7f7550b056249b48c4c8f0b83bf52694@gmail.com"},"message":"[ASTImporter][Structural Eq] Check for isBeingDefined\n\nSummary:\nIf one definition is currently being defined, we do not compare for\nequality and we assume that the decls are equal.\n\nReviewers: a_sidorin\n\nSubscribers: rnkovacs, dkrupp, Szelethus, cfe-commits\n\nDifferential Revision: https://reviews.llvm.org/D53697\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345760 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/3c95e03870283af2773cb8050040c342af18708b"},{"sha":"62ba7037896b1245254adb4e6c4cdb04ef29e022","author":{"name":"Richard Smith","email":"9d938db41bf7d07f88e8d8adba89b477eaafa4d6@metafoo.co.uk"},"message":"Part of PR39508: Emit an @llvm.invariant.start after storing to\n__tls_guard.\n\n__tls_guard can only ever transition from 0 to 1, and only once. This\npermits LLVM to remove repeated checks for TLS initialization and\nrepeated initialization code in cases like:\n\n  int g();\n  thread_local int n = g();\n  int a = n + n;\n\nwhere we could not prove that __tls_guard was still 'true' when checking\nit for the second reference to 'n' in the initializer of 'a'.\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345774 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/62ba7037896b1245254adb4e6c4cdb04ef29e022"},{"sha":"ed32d216e99c57ee7cd0802a16e3f667850a4b43","author":{"name":"Eli Friedman","email":"8e5ee4945af1f17b38864e236876caaaabff2d25@codeaurora.org"},"message":"[ARM64] [Windows] Implement _InterlockedExchangeAdd*_* builtins.\n\nThese apparently need to be proper builtins to handle the Windows\nSDK.\n\nDifferential Revision: https://reviews.llvm.org/D53916\n\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345779 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/ed32d216e99c57ee7cd0802a16e3f667850a4b43"},{"sha":"d81da87531b6cd4829b939f433845b60ec1e00d6","author":{"name":"Eli Friedman","email":"8e5ee4945af1f17b38864e236876caaaabff2d25@codeaurora.org"},"message":"[AArch64] [Windows] Emit unwind tables by default.\n\nUnwind tables are necessary even in code that doesn't support\nexceptions.  The tables are used for setjmp(), and by debuggers.\n\nDifferential Revision: https://reviews.llvm.org/D53618\n\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345781 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/d81da87531b6cd4829b939f433845b60ec1e00d6"},{"sha":"31b15cc64474f7142df204f25ac6cb834d1501de","author":{"name":"Shafik Yaghmour","email":"6e9e3d750faed5ef3ef2ec5a27994e1ab82a0117@apple.com"},"message":"Revert \"[ASTImporter][Structural Eq] Check for isBeingDefined\"\n\nThis reverts commit r345760\n\nbecause it caused an assertion in the lldb test suite. This is the log from the build bot: http://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/12003/\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345784 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/31b15cc64474f7142df204f25ac6cb834d1501de"},{"sha":"619111f5ccf349b635e4987ec02d15777c571495","author":{"name":"Mandeep Singh Grang","email":"414d85aef57abf2205af5f15419474bebe846edf@codeaurora.org"},"message":"[COFF, ARM64] Change setjmp for AArch64 Windows to use Intrinsic.sponentry\n\nSummary: ARM64 setjmp expects sp on entry instead of framepointer.\n\nReviewers: mgrang, rnk, TomTan, compnerd, mstorsjo, efriedma\n\nReviewed By: mstorsjo\n\nSubscribers: javed.absar, kristof.beyls, chrib, cfe-commits\n\nDifferential Revision: https://reviews.llvm.org/D53684\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345792 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/619111f5ccf349b635e4987ec02d15777c571495"},{"sha":"66e7c70fc54b902067dfa0132838d82dee7e8588","author":{"name":"Artem Dergachev","email":"6cbf4556c0fbd074258a67000a54f5a83515d78c@gmail.com"},"message":"[analyzer] pr39348: MallocChecker: Realize that sized delete isn't custom delete.\n\nMallocChecker no longer thinks that operator delete() that accepts the size of\nthe object to delete (available since C++14 or under -fsized-deallocation)\nis some weird user-defined operator. Instead, it handles it like normal delete.\n\nAdditionally, it exposes a regression in NewDelete-intersections.mm's\ntestStandardPlacementNewAfterDelete() test, where the diagnostic is delayed\nfrom before the call of placement new into the code of placement new\nin the header. This happens because the check for pass-into-function-after-free\nfor placement arguments is located in checkNewAllocator(), which happens after\nthe allocator is inlined, which is too late. Move this use-after-free check\ninto checkPreCall instead, where it works automagically because the guard\nthat prevents it from working is useless and can be removed as well.\n\nThis commit causes regressions under -analyzer-config\nc++-allocator-inlining=false but this option is essentially unsupported\nbecause the respective feature has been enabled by default quite a while ago.\n\nDifferential Revision: https://reviews.llvm.org/D53543\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345802 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/66e7c70fc54b902067dfa0132838d82dee7e8588"},{"sha":"c6a35f056e82c4eb18eb07e38ae2cb60858bebd2","author":{"name":"Richard Smith","email":"9d938db41bf7d07f88e8d8adba89b477eaafa4d6@metafoo.co.uk"},"message":"Fix regression in behavior of clang -x c++-header -fmodule-name=XXX\n-fsyntax-only.\n\nThe driver accidentally stopped passing the input filenames on to -cc1\nin this mode due to confusion over what action was being requested.\n\nThis change also fixes a couple of crashes I encountered when passing\nmultiple files to such a -cc1 invocation.\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345803 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/c6a35f056e82c4eb18eb07e38ae2cb60858bebd2"},{"sha":"9d42cf475d9e676c98b98738791c24ce6c84bcb8","author":{"name":"Thomas Lively","email":"6d0eb04eded5b535c7fb3ea181fdac9b5e798a46@google.com"},"message":"[WebAssembly] Fix type names in truncation builtins\n\nSummary: Use the same convention as all the other WebAssembly builtin names.\n\nReviewers: aheejin, dschuff\n\nSubscribers: sbc100, jgravelle-google, sunfish, kristina, cfe-commits\n\nDifferential Revision: https://reviews.llvm.org/D53724\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345804 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/9d42cf475d9e676c98b98738791c24ce6c84bcb8"},{"sha":"016d569f8681ff3e691cb995451d0e33a9245ccc","author":{"name":"Richard Smith","email":"9d938db41bf7d07f88e8d8adba89b477eaafa4d6@metafoo.co.uk"},"message":"Fix typo in comment.\n\ngit-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345805 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/UofT-EcoSystem/clang/commits/016d569f8681ff3e691cb995451d0e33a9245ccc"}]}
{"actor":{"display_login":"qiongsiwu"}}