Re: r310403 - Thread Safety Analysis: warn on nonsensical attributes.

It causes failure in check-libcxx with just-built clang. http://bb.pgr.jp/builders/bootstrap-clang-libcxx-lld-i686-linux/builds/758 On Wed, Aug 9, 2017 at 4:45 AM Josh Gao via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: jmgao > Date: Tue Aug 8 12:44:35 2017 > New Revision: 310403

Re: r310403 - Thread Safety Analysis: warn on nonsensical attributes.

Sorry for the breakage, I'm reverting the patch that caused this now. On Fri, Aug 11, 2017 at 12:36 AM, NAKAMURA Takumi wrote: > It causes failure in check-libcxx with just-built clang. > > http://bb.pgr.jp/builders/bootstrap-clang-libcxx-lld-i686-linux/builds/758 > > > On Wed, Aug 9, 2017 at 4:

r310698 - Revert "Thread Safety Analysis: warn on nonsensical attributes."

Author: jmgao Date: Fri Aug 11 00:54:35 2017 New Revision: 310698 URL: http://llvm.org/viewvc/llvm-project?rev=310698&view=rev Log: Revert "Thread Safety Analysis: warn on nonsensical attributes." This reverts commit rL310403, which caused spurious warnings in libc++, because it didn't properly h

Re: r310403 - Thread Safety Analysis: warn on nonsensical attributes.

Reverted in r310698. On Fri, Aug 11, 2017 at 12:51 AM, Josh Gao wrote: > Sorry for the breakage, I'm reverting the patch that caused this now. > > On Fri, Aug 11, 2017 at 12:36 AM, NAKAMURA Takumi > wrote: > >> It causes failure in check-libcxx with just-built clang. >> >> http://bb.pgr.jp/buil

[PATCH] D35533: [Basic] Update CMakeLists.txt to handle Repo

minseong.kim added a comment. kindly ping~ https://reviews.llvm.org/D35533 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r310403 - Thread Safety Analysis: warn on nonsensical attributes.

Thanks! On Fri, Aug 11, 2017 at 4:56 PM Josh Gao wrote: > Reverted in r310698. > > On Fri, Aug 11, 2017 at 12:51 AM, Josh Gao wrote: > >> Sorry for the breakage, I'm reverting the patch that caused this now. >> >> On Fri, Aug 11, 2017 at 12:36 AM, NAKAMURA Takumi >> wrote: >> >>> It causes fai

[PATCH] D36261: [clangd] Use multiple working threads in clangd.

klimek accepted this revision. klimek added a comment. This revision is now accepted and ready to land. lg https://reviews.llvm.org/D36261 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-com

[PATCH] D36398: [clangd] Check if CompileCommand has changed on forceReparse.

klimek accepted this revision. klimek added a comment. This revision is now accepted and ready to land. lg Comment at: unittests/clangd/ClangdTests.cpp:509-510 + /*RunSynchronously=*/true); + // No need to sync reparses, because RunSynchronously is set +

Re: r310401 - PR19668, PR23034: Fix handling of move constructors and deleted copy

Hi guys, I'm SO sorry about the delays. I've been having all sorts of trouble getting that debug build on the board (from ld running out of memory to the board just crashing on me, in which case I need to ask someone else to reboot it because I can't power cycle it remotely). I can assure you this

[PATCH] D35955: clang-format: Add preprocessor directive indentation

klimek added a comment. LG from my side. https://reviews.llvm.org/D35955 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D36572: Implemented P0409R2 - Allow lambda capture [=, this]

hamzasood added inline comments. Comment at: test/FixIt/fixit-cxx0x.cpp:57 (void)[&, &i, &i]{}; // expected-error 2{{'&' cannot precede a capture when the capture default is '&'}} - (void)[=, this]{ this->g(5); }; // expected-error{{'this' cannot be explicitly captured}}

r310700 - [Bash-autocompletion] Add --autocomplete flag to 5.0 release notes

Author: yamaguchi Date: Fri Aug 11 02:44:42 2017 New Revision: 310700 URL: http://llvm.org/viewvc/llvm-project?rev=310700&view=rev Log: [Bash-autocompletion] Add --autocomplete flag to 5.0 release notes Summary: I thought we should add this information to release notes, because we added a new fla

[PATCH] D36567: [Bash-autocompletion] Add --autocomplete flag to 5.0 release notes

This revision was automatically updated to reflect the committed changes. Closed by commit rL310700: [Bash-autocompletion] Add --autocomplete flag to 5.0 release notes (authored by yamaguchi). Changed prior to commit: https://reviews.llvm.org/D36567?vs=110518&id=110679#toc Repository: rL LLV

[PATCH] D30465: [mips] Set the Int64Type / IntMaxType types correctly for OpenBSD/mips64

sdardis closed this revision. sdardis added a comment. Closing this, https://reviews.llvm.org/rL297098. Repository: rL LLVM https://reviews.llvm.org/D30465 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mai

[PATCH] D36397: [clangd] Fixed a data race.

klimek added inline comments. Comment at: clangd/ClangdUnit.cpp:714 -void CppFile::cancelRebuilds() { +void CppFile::cancelRebuild() { deferCancelRebuild().get(); } + Somewhat orthogonal question - why CppFile? We do support other languages in Clangd, right? (

[PATCH] D36397: [clangd] Fixed a data race.

ilya-biryukov added inline comments. Comment at: clangd/ClangdUnit.cpp:714 -void CppFile::cancelRebuilds() { +void CppFile::cancelRebuild() { deferCancelRebuild().get(); } + klimek wrote: > Somewhat orthogonal question - why CppFile? We do support other languag

[PATCH] D36184: [clang-diff] Filter AST nodes

johannes updated this revision to Diff 110687. johannes added a comment. clarify test/Tooling/clang-diff-ast.cpp https://reviews.llvm.org/D36184 Files: lib/Tooling/ASTDiff/ASTDiff.cpp test/Tooling/clang-diff-ast.cpp test/Tooling/clang-diff-json.cpp Index: test/Tooling/clang-diff-json.cpp

[PATCH] D36185: [clang-diff] Fix similarity computation

johannes updated this revision to Diff 110688. johannes added a comment. newline in error message https://reviews.llvm.org/D36185 Files: include/clang/Tooling/ASTDiff/ASTDiff.h lib/Tooling/ASTDiff/ASTDiff.cpp test/Tooling/clang-diff-bottomup.cpp test/Tooling/clang-diff-opt.cpp test/To

r310702 - Revert r310057

Author: smaksimovic Date: Fri Aug 11 04:03:54 2017 New Revision: 310702 URL: http://llvm.org/viewvc/llvm-project?rev=310702&view=rev Log: Revert r310057 Bring back changes which r304953 introduced since they were in fact not the cause of failures described in r310057 commit message. Added: c

r310704 - Revert r302670 for the upcoming 5.0.0 release

Author: smaksimovic Date: Fri Aug 11 04:39:07 2017 New Revision: 310704 URL: http://llvm.org/viewvc/llvm-project?rev=310704&view=rev Log: Revert r302670 for the upcoming 5.0.0 release This is causing failures when compiling clang with -O3 as one of the structures used by clang is passed by value

r310706 - [modules] Set the lexical DC for dummy tag decls that refer to hidden

Author: arphaman Date: Fri Aug 11 05:06:52 2017 New Revision: 310706 URL: http://llvm.org/viewvc/llvm-project?rev=310706&view=rev Log: [modules] Set the lexical DC for dummy tag decls that refer to hidden declarations that are made visible after the dummy is parsed and ODR verified Prior to this

Re: r310706 - [modules] Set the lexical DC for dummy tag decls that refer to hidden

Hi Hans & Richard, Is it possible to get this merged into LLVM 5.0? Cheers, Alex On 11 August 2017 at 13:06, Alex Lorenz via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: arphaman > Date: Fri Aug 11 05:06:52 2017 > New Revision: 310706 > > URL: http://llvm.org/viewvc/llvm-project?r

[clang-tools-extra] r310707 - Implement hicpp-braces-around-statements as an alias to readability-braces-around-statements.

Author: aaronballman Date: Fri Aug 11 05:12:36 2017 New Revision: 310707 URL: http://llvm.org/viewvc/llvm-project?rev=310707&view=rev Log: Implement hicpp-braces-around-statements as an alias to readability-braces-around-statements. Patch by Jonas Toth. Added: clang-tools-extra/trunk/docs/

[PATCH] D36354: [clang-tidy] Implement type-based check for `gsl::owner`

aaron.ballman added inline comments. Comment at: clang-tidy/cppcoreguidelines/OwningMemoryCheck.cpp:149 + // Deletion of non-owners, with `delete variable;` + if (DeletedVariable != nullptr) { +assert(DeleteStmt != nullptr && Can `DeletedVariable` ever be n

[PATCH] D36610: [Tooling] Add option to getFullyQualifiedName using a custom PritingPolicy

mikhail.ramalho created this revision. Herald added a subscriber: klimek. In our user case, we need to generate unique names for every tag, including anonymous structs/unions. This adds a custom PrintingPolicy option to getFullyQualifiedName and the test case shows that different names are gene

[PATCH] D36611: Implemented P0683R1 - Default member initializers for bit-fields

hamzasood created this revision. This patch implements P0683R1 . Member initialisers are allowed pre-C++11 as an extension. So I've also allowed bitfield member initialisers pre-C++2a as an extension (with appropriate warnings) for consistency. https://reviews.llvm.o

Re: r310401 - PR19668, PR23034: Fix handling of move constructors and deleted copy

Hi again, I finally got the debug build, but unfortunately the stack traces that the tests print look the same. My suspicion is that this is because the addresses printed by the tests are funny (i.e. odd numbers instead of divisible by 4). I tried to follow those addresses in an objdump of the exe

[PATCH] D36610: [Tooling] Add option to getFullyQualifiedName using a custom PritingPolicy

arphaman added inline comments. Comment at: unittests/Tooling/QualTypeNamesTest.cpp:247 + "struct (anonymous struct at input.cc:1:1)"; + PrintingPolicy.runOver( + "struct\n" You should probably use multiline string here, i.e. R(" ... ") https://revie

[PATCH] D36354: [clang-tidy] Implement type-based check for `gsl::owner`

JonasToth added inline comments. Comment at: clang-tidy/cppcoreguidelines/OwningMemoryCheck.cpp:149 + // Deletion of non-owners, with `delete variable;` + if (DeletedVariable != nullptr) { +assert(DeleteStmt != nullptr && aaron.ballman wrote: > Can `Deleted

[PATCH] D35941: Fix -Wshadow false positives with function-local classes.

alexfh added a comment. In https://reviews.llvm.org/D35941#838633, @rsmith wrote: > In https://reviews.llvm.org/D35941#823524, @alexfh wrote: > > > IIUC, most cases where -Wshadow warnings are issued is when a declaration > > from an enclosing scope would be accessible if there was no declaratio

Re: r310401 - PR19668, PR23034: Fix handling of move constructors and deleted copy

That's really strange. It looks like some random behavior. Did you run some memory checker like valgrind? Do the environment provided by the test runner and yours match? Sent from my phone. Please excuse my brevity. > On 11 Aug 2017, at 15:58, Diana Picus wrote: > > Hi again, > > I finally g

Re: r310401 - PR19668, PR23034: Fix handling of move constructors and deleted copy

Well, these are ASAN tests, I'm not sure how that would interact with Valgrind. Anyway, I'll try to reproduce the environment, I'm guessing it would be best to catch this in gdb so I can actually see what's going on. On 11 August 2017 at 15:21, Vassil Vassilev wrote: > That's really strange. It l

[PATCH] D34158: For Linux/gnu compatibility, preinclude if the file is available

mibintc planned changes to this revision. mibintc added a comment. Need TO FIX: We should be stripping the new arg as well: add "-fsystem-include-if-exists" argument to the list of include things in the skipArgs() function in lib/Driver/Job.cpp https://reviews.llvm.org/D34158 __

[PATCH] D36397: [clangd] Fixed a data race.

ilya-biryukov added a comment. A follow-up after discussion offline: we should make an attempt to simplify the threading code in `ClangdUnit.h` by moving the async worker loop of `CppFile` into the class itself, rather than relying on external scheduling to do the right thing. @klimek, are ther

[PATCH] D36614: [clang-format] Refine trailing comment detection

krasimir created this revision. Herald added a subscriber: klimek. This patch fixes an non-idempotency issue connected with detection of trailing comments. Consider formatting the following code with column limit at `V`: V const /* comment comment */ A = B; The comment is

[PATCH] D36327: [OpenCL] Allow targets emit optimized pipe functions for power of 2 type sizes

yaxunl added a comment. John, do you have any comments? Thanks. https://reviews.llvm.org/D36327 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D35370: [clang-tidy] Add a close-on-exec check on inotify_init() in Android module.

hokein added inline comments. Comment at: clang-tidy/android/CloexecInotifyInitCheck.cpp:27 + replaceFunc(Result, + "prefer inotify_init() to inotify_init1() " + "because inotify_init1() allows IN_CLOEXEC", The parameter 2 and 3 are not

[PATCH] D35368: [clang-tidy] Add a close-on-exec check on inotify_init1() in Android module.

hokein added inline comments. Comment at: clang-tidy/android/CloexecInotifyInit1Check.cpp:28 +void CloexecInotifyInit1Check::check(const MatchFinder::MatchResult &Result) { + insertMacroFlag(Result, "IN_CLOEXEC", /*ArgPos=*/0); +} nit: `/*MarcoFlag=*/` ===

[PATCH] D35364: [clang-tidy] Add a close-on-exec check on dup() in Android module.

hokein added inline comments. Comment at: clang-tidy/android/CloexecCheck.h:91 + /// Helper function to get the spelling of a particular argument. + StringRef getSpellingArg(const ast_matchers::MatchFinder::MatchResult &Result, + int N) const; ---

[PATCH] D34170: [libcxx] Moving compiler specific test infrastructure to compiler.py

bcraig added a comment. ping https://reviews.llvm.org/D34170 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D32411: [libcxx] Provide #include_next alternative for MSVC

bcraig added a comment. ping https://reviews.llvm.org/D32411 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D35624: Removal of microMIPS64R6

sdardis accepted this revision. sdardis added a comment. This revision is now accepted and ready to land. One nit inlined, you can address it when committing. Comment at: Basic/Targets/Mips.cpp:206 bool MipsTargetInfo::validateTarget(DiagnosticsEngine &Diags) const { + if ((g

[PATCH] D35982: [mips] Introducing option -mabs=[legacy/2008]

sdardis requested changes to this revision. sdardis added a comment. This revision now requires changes to proceed. Follow @atanasyan 's suggestion and post a patch that renames that enum along with other changes. One comment inlined. Comment at: clang/Basic/DiagnosticDriverK

r310714 - [mips] Support implicit gpopt with N64 when using -fno-pic

Author: sdardis Date: Fri Aug 11 08:01:34 2017 New Revision: 310714 URL: http://llvm.org/viewvc/llvm-project?rev=310714&view=rev Log: [mips] Support implicit gpopt with N64 when using -fno-pic As clang defaults to -mno-abicalls when using -fno-pic for N64, implicitly use -mgpopt in that case. Re

[PATCH] D36315: [mips] Support implicit gpopt with N64 when using -fno-pic

This revision was automatically updated to reflect the committed changes. Closed by commit rL310714: [mips] Support implicit gpopt with N64 when using -fno-pic (authored by sdardis). Repository: rL LLVM https://reviews.llvm.org/D36315 Files: cfe/trunk/lib/Driver/ToolChains/Clang.cpp cfe/t

[PATCH] D36315: [mips] Support implicit gpopt with N64 when using -fno-pic

sdardis added a comment. Thanks for the review and spotting this bug that lead me to finding the other bug. Repository: rL LLVM https://reviews.llvm.org/D36315 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin

[PATCH] D36354: [clang-tidy] Implement type-based check for `gsl::owner`

JonasToth updated this revision to Diff 110730. JonasToth marked 6 inline comments as done. JonasToth added a comment. - order release notes - address aarons comments - adjust diagnostic for function arguments https://reviews.llvm.org/D36354 Files: clang-tidy/cppcoreguidelines/CMakeLists.txt

[PATCH] D36354: [clang-tidy] Implement type-based check for `gsl::owner`

JonasToth marked an inline comment as done. JonasToth added inline comments. Comment at: clang-tidy/cppcoreguidelines/OwningMemoryCheck.cpp:188 +diag(OwnerAssignment->getLocStart(), + "assigning neither an owner nor a recognized resource") +<< SourceRange(Owne

[PATCH] D36354: [clang-tidy] Implement type-based check for `gsl::owner`

JonasToth updated this revision to Diff 110731. JonasToth added a comment. remove manual source range creation, where i forgot it before https://reviews.llvm.org/D36354 Files: clang-tidy/cppcoreguidelines/CMakeLists.txt clang-tidy/cppcoreguidelines/CppCoreGuidelinesTidyModule.cpp clang-ti

r310715 - [mips] Add missing mips-registered-target to mips test.

Author: sdardis Date: Fri Aug 11 08:23:23 2017 New Revision: 310715 URL: http://llvm.org/viewvc/llvm-project?rev=310715&view=rev Log: [mips] Add missing mips-registered-target to mips test. Modified: cfe/trunk/test/Driver/mips-abi.c Modified: cfe/trunk/test/Driver/mips-abi.c URL: http://llv

[PATCH] D36184: [clang-diff] Filter AST nodes

arphaman accepted this revision. arphaman added a comment. This revision is now accepted and ready to land. LGTM. Comment at: test/Tooling/clang-diff-ast.cpp:68 +// nodes from other files are excluded +// CHECK-NOT {{.}} +#include "clang-diff-ast.cpp" johannes

[PATCH] D35948: [CommonOptionsParser] Expose ArgumentsAdjustingCompilationDatabase

arphaman added a reviewer: klimek. arphaman added a comment. @klimek Ping. https://reviews.llvm.org/D35948 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D36186: [clang-diff] Improve and test getNodeValue

arphaman added inline comments. Comment at: lib/Tooling/ASTDiff/ASTDiff.cpp:428 +Value += getRelativeName(V) + "(" + V->getType().getAsString(TypePP) + ")"; +if (auto *C = dyn_cast(D)) { + for (auto *Init : C->inits()) { It looks like you're doing a

Re: r310691 - PR33489: A function-style cast to a deduced class template specialization type is type-dependent if it can't be resolved due to a type-dependent argument.

Thanks! r310719. On Thu, Aug 10, 2017 at 7:08 PM, Richard Smith wrote: > Hi Hans, this fixes a bug you wanted to treat as a 5.0 release blocker, so > should be ported to the branch :) > > On 10 August 2017 at 19:04, Richard Smith via cfe-commits > wrote: >> >> Author: rsmith >> Date: Thu Aug 10

[PATCH] D29660: [OpenMP] Add flag for overwriting default PTX version for OpenMP targets

gtbercea added a comment. I have re-enabled the previous offloading tests and moved the new GPU offloading tests to a new file which is disabled for linux (for now). 310718 Alex thanks so much for the logs, they have been very useful to understand what's going on. Aleksey, I have since tried

Re: r310692 - PR33850: Update cxx_dr_status for Clang 5 branch.

Merged to 5.0 in r310720. On Thu, Aug 10, 2017 at 8:14 PM, Richard Smith via cfe-commits wrote: > Author: rsmith > Date: Thu Aug 10 20:14:20 2017 > New Revision: 310692 > > URL: http://llvm.org/viewvc/llvm-project?rev=310692&view=rev > Log: > PR33850: Update cxx_dr_status for Clang 5 branch. > >

Re: r310694 - Implement latest feature test macro recommendations, P0096R4.

Thanks! I've merged r309054 in r310721 to make it apply cleanly, and this one in r310722. On Thu, Aug 10, 2017 at 8:42 PM, Richard Smith wrote: > Hi Hans, this should go onto the Clang 5 branch. Thanks! > > On 10 August 2017 at 20:39, Richard Smith via cfe-commits > wrote: >> >> Author: rsmith >

Re: r310700 - [Bash-autocompletion] Add --autocomplete flag to 5.0 release notes

Merged to 5.0 in r310723. Thanks! On Fri, Aug 11, 2017 at 2:44 AM, Yuka Takahashi via cfe-commits wrote: > Author: yamaguchi > Date: Fri Aug 11 02:44:42 2017 > New Revision: 310700 > > URL: http://llvm.org/viewvc/llvm-project?rev=310700&view=rev > Log: > [Bash-autocompletion] Add --autocomplete f

[PATCH] D36610: [Tooling] Add option to getFullyQualifiedName using a custom PritingPolicy

saugustine accepted this revision. saugustine added a comment. This revision is now accepted and ready to land. This is a good change as far as functionality, but I defer to others on the style and other details. https://reviews.llvm.org/D36610 ___

[PATCH] D36567: [Bash-autocompletion] Add --autocomplete flag to 5.0 release notes

ruiu added a comment. Yuka, You seem to have committed a release note for 5.0 to SVN trunk. What you should've done is to do that to the 5.0 branch. I'll correct the error for you. Repository: rL LLVM https://reviews.llvm.org/D36567 ___ cfe-com

[PATCH] D36567: [Bash-autocompletion] Add --autocomplete flag to 5.0 release notes

ruiu added a comment. Hans already merged it in r310723. Repository: rL LLVM https://reviews.llvm.org/D36567 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D36610: [Tooling] Add option to getFullyQualifiedName using a custom PritingPolicy

mikhail.ramalho added a comment. Should I update the other tests in QualTypeNamesTest.cpp to use multiline strings as well? https://reviews.llvm.org/D36610 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailma

[clang-tools-extra] r310727 - Add hicpp-exception-baseclass to the HIC++ module.

Author: aaronballman Date: Fri Aug 11 09:31:51 2017 New Revision: 310727 URL: http://llvm.org/viewvc/llvm-project?rev=310727&view=rev Log: Add hicpp-exception-baseclass to the HIC++ module. This enforces that throwing an exception in C++ requires that exception to inherit from std::exception. P

Re: r310704 - Revert r302670 for the upcoming 5.0.0 release

Merged to 5.0 in r310728. On Fri, Aug 11, 2017 at 4:39 AM, Stefan Maksimovic via cfe-commits wrote: > Author: smaksimovic > Date: Fri Aug 11 04:39:07 2017 > New Revision: 310704 > > URL: http://llvm.org/viewvc/llvm-project?rev=310704&view=rev > Log: > Revert r302670 for the upcoming 5.0.0 release

[clang-tools-extra] r310732 - Enable exceptions for this test case to speculatively fix the build bots.

Author: aaronballman Date: Fri Aug 11 09:46:45 2017 New Revision: 310732 URL: http://llvm.org/viewvc/llvm-project?rev=310732&view=rev Log: Enable exceptions for this test case to speculatively fix the build bots. Hopefully corrects: http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps

[PATCH] D35370: [clang-tidy] Add a close-on-exec check on inotify_init() in Android module.

yawanng updated this revision to Diff 110749. yawanng marked an inline comment as done. https://reviews.llvm.org/D35370 Files: clang-tidy/android/AndroidTidyModule.cpp clang-tidy/android/CMakeLists.txt clang-tidy/android/CloexecInotifyInitCheck.cpp clang-tidy/android/CloexecInotifyInitChe

[PATCH] D29660: [OpenMP] Add flag for overwriting default PTX version for OpenMP targets

alekseyshl added a comment. In https://reviews.llvm.org/D29660#839209, @gtbercea wrote: > I have re-enabled the previous offloading tests and moved the new GPU > offloading tests to a new file which is disabled for linux (for now). > > 310718 > > Alex thanks so much for the logs, they have been

[PATCH] D35367: [clang-tidy] Add a close-on-exec check on epoll_create() in Android module.

yawanng updated this revision to Diff 110751. https://reviews.llvm.org/D35367 Files: clang-tidy/android/AndroidTidyModule.cpp clang-tidy/android/CMakeLists.txt clang-tidy/android/CloexecEpollCreateCheck.cpp clang-tidy/android/CloexecEpollCreateCheck.h docs/ReleaseNotes.rst docs/clang-

[PATCH] D35368: [clang-tidy] Add a close-on-exec check on inotify_init1() in Android module.

yawanng updated this revision to Diff 110750. yawanng marked 2 inline comments as done. https://reviews.llvm.org/D35368 Files: clang-tidy/android/AndroidTidyModule.cpp clang-tidy/android/CMakeLists.txt clang-tidy/android/CloexecInotifyInit1Check.cpp clang-tidy/android/CloexecInotifyInit1C

[PATCH] D35365: [clang-tidy] Add a close-on-exec check on epoll_create1() in Android module.

yawanng updated this revision to Diff 110752. https://reviews.llvm.org/D35365 Files: clang-tidy/android/AndroidTidyModule.cpp clang-tidy/android/CMakeLists.txt clang-tidy/android/CloexecEpollCreate1Check.cpp clang-tidy/android/CloexecEpollCreate1Check.h docs/ReleaseNotes.rst docs/clan

[PATCH] D35364: [clang-tidy] Add a close-on-exec check on dup() in Android module.

yawanng updated this revision to Diff 110753. yawanng marked an inline comment as done. https://reviews.llvm.org/D35364 Files: clang-tidy/android/AndroidTidyModule.cpp clang-tidy/android/CMakeLists.txt clang-tidy/android/CloexecCheck.cpp clang-tidy/android/CloexecCheck.h clang-tidy/andr

[PATCH] D35364: [clang-tidy] Add a close-on-exec check on dup() in Android module.

yawanng added inline comments. Comment at: clang-tidy/android/CloexecCheck.h:91 + /// Helper function to get the spelling of a particular argument. + StringRef getSpellingArg(const ast_matchers::MatchFinder::MatchResult &Result, + int N) const; --

[PATCH] D35363: [clang-tidy] Add a close-on-exec check on accept4() in Android module.

yawanng updated this revision to Diff 110755. https://reviews.llvm.org/D35363 Files: clang-tidy/android/AndroidTidyModule.cpp clang-tidy/android/CMakeLists.txt clang-tidy/android/CloexecAccept4Check.cpp clang-tidy/android/CloexecAccept4Check.h docs/ReleaseNotes.rst docs/clang-tidy/che

[PATCH] D35362: [clang-tidy] Add a close-on-exec check on accept() in Android module.

yawanng updated this revision to Diff 110758. https://reviews.llvm.org/D35362 Files: clang-tidy/android/AndroidTidyModule.cpp clang-tidy/android/CMakeLists.txt clang-tidy/android/CloexecAcceptCheck.cpp clang-tidy/android/CloexecAcceptCheck.h docs/ReleaseNotes.rst docs/clang-tidy/check

r310733 - Add -fsanitize=fuzzer-no-link flag to the driver.

Author: george.karpenkov Date: Fri Aug 11 10:22:58 2017 New Revision: 310733 URL: http://llvm.org/viewvc/llvm-project?rev=310733&view=rev Log: Add -fsanitize=fuzzer-no-link flag to the driver. The flag will perform instrumentation necessary to the fuzzing, but will NOT link libLLVMFuzzer.a librar

[PATCH] D36572: Implemented P0409R2 - Allow lambda capture [=, this]

rjmccall added inline comments. Comment at: test/FixIt/fixit-cxx0x.cpp:57 (void)[&, &i, &i]{}; // expected-error 2{{'&' cannot precede a capture when the capture default is '&'}} - (void)[=, this]{ this->g(5); }; // expected-error{{'this' cannot be explicitly captured}}

[PATCH] D36580: [OpenCL] Support variable memory scope in atomic builtins

rjmccall added inline comments. Comment at: include/clang/Basic/SyncScope.h:47 + return Scopes; +} + You could just return an ArrayRef to a static const array. Comment at: lib/CodeGen/CGAtomic.cpp:687 + + auto *SC = Builder.CreateIntCast(Scop

[PATCH] D36572: Implemented P0409R2 - Allow lambda capture [=, this]

hamzasood updated this revision to Diff 110776. hamzasood added a comment. @rjmccall The warning is emitted, but you're right that there's no test to ensure that actually happens; test/CXX/expr/expr.prim/expr.prim.lambda/p8.cpp used Wno-c++2a-extensions to suppress the warning. I've changed it

[PATCH] D36354: [clang-tidy] Implement type-based check for `gsl::owner`

aaron.ballman added inline comments. Comment at: clang-tidy/cppcoreguidelines/OwningMemoryCheck.cpp:188 +diag(OwnerAssignment->getLocStart(), + "assigning neither an owner nor a recognized resource") +<< SourceRange(OwnerAssignment->getLocStart(),

[PATCH] D29660: [OpenMP] Add flag for overwriting default PTX version for OpenMP targets

alekseyshl added a comment. http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/7109 failed on r310718, please fix. Repository: rL LLVM https://reviews.llvm.org/D29660 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http

[PATCH] D36635: Add a Dockerfile for clang-proto-fuzzer

kcc created this revision. Add a Dockerfile for clang-proto-fuzzer https://reviews.llvm.org/D36635 Files: tools/clang-fuzzer/Dockerfile tools/clang-fuzzer/README.txt Index: tools/clang-fuzzer/README.txt === --- tools/clang-fu

Re: r309226 - Headers: improve ARM EHABI coverage of unwind.h

Hi, I've noticed that the code in compiler-rt/lib/builtins/gcc_personality_v0.c refers to _Unwind_Exception as "struct _Unwind_Exception". With this change, it is not a struct anymore on ARM. Should that code be fixed, or is it a problem in this change? compiler-rt/lib/builtins/gcc_personality_v0

[PATCH] D36354: [clang-tidy] Implement type-based check for `gsl::owner`

JonasToth marked 5 inline comments as done. JonasToth added inline comments. Comment at: clang-tidy/cppcoreguidelines/OwningMemoryCheck.cpp:230 +<< BadOwnerInitialization->getSourceRange(); +// FIXME FixitHint to rewrite the type if possible + aaron.b

[PATCH] D36354: [clang-tidy] Implement type-based check for `gsl::owner`

JonasToth updated this revision to Diff 110799. JonasToth marked an inline comment as done. JonasToth added a comment. address review comments from aaron https://reviews.llvm.org/D36354 Files: clang-tidy/cppcoreguidelines/CMakeLists.txt clang-tidy/cppcoreguidelines/CppCoreGuidelinesTidyModu

[PATCH] D29660: [OpenMP] Add flag for overwriting default PTX version for OpenMP targets

gtbercea added a comment. In https://reviews.llvm.org/D29660#839275, @alekseyshl wrote: > In https://reviews.llvm.org/D29660#839209, @gtbercea wrote: > > > I have re-enabled the previous offloading tests and moved the new GPU > > offloading tests to a new file which is disabled for linux (for no

[PATCH] D36580: [OpenCL] Support variable memory scope in atomic builtins

t-tye added inline comments. Comment at: include/clang/Basic/SyncScope.h:46 + Scopes.push_back(SyncScope::OpenCLSubGroup); + return Scopes; +} Should there be an assert/static_assert in case SyncScope enum grows due to other languages? Comme

[libcxx] r310758 - [libcxx] [test] Rename _Tp to T. NFCI.

Author: stl_msft Date: Fri Aug 11 13:53:53 2017 New Revision: 310758 URL: http://llvm.org/viewvc/llvm-project?rev=310758&view=rev Log: [libcxx] [test] Rename _Tp to T. NFCI. This improves readability and (theoretically) improves portability, as _Ugly names are reserved. Modified: libcxx/tru

[libcxx] r310759 - [libcxx] [test] Rename __c to ch. NFCI.

Author: stl_msft Date: Fri Aug 11 13:54:01 2017 New Revision: 310759 URL: http://llvm.org/viewvc/llvm-project?rev=310759&view=rev Log: [libcxx] [test] Rename __c to ch. NFCI. This improves readability and (theoretically) improves portability, as __ugly names are reserved. Modified: libcxx/t

[libcxx] r310761 - [libcxx] [test] Rename _Up to U, etc. NFCI.

Author: stl_msft Date: Fri Aug 11 13:54:09 2017 New Revision: 310761 URL: http://llvm.org/viewvc/llvm-project?rev=310761&view=rev Log: [libcxx] [test] Rename _Up to U, etc. NFCI. This improves readability and (theoretically) improves portability, as _Ugly names are reserved. This performs additi

[libcxx] r310760 - [libcxx] [test] Rename __x to x. NFCI.

Author: stl_msft Date: Fri Aug 11 13:54:06 2017 New Revision: 310760 URL: http://llvm.org/viewvc/llvm-project?rev=310760&view=rev Log: [libcxx] [test] Rename __x to x. NFCI. This improves readability and (theoretically) improves portability, as __ugly names are reserved. Modified: libcxx/tr

[PATCH] D36635: Add a Dockerfile for clang-proto-fuzzer

morehouse added inline comments. Comment at: tools/clang-fuzzer/Dockerfile:22 +# Get LLVM +RUN svn co http://llvm.org/svn/llvm-project/llvm/trunk llvm +RUN cd llvm/tools && svn co http://llvm.org/svn/llvm-project/cfe/trunk clang -r $(cd ../ && svn info | grep Revision | awk '{pr

[PATCH] D36635: Add a Dockerfile for clang-proto-fuzzer

kcc updated this revision to Diff 110810. kcc added a comment. fix 'svn co' command (apparently it did not matter though) https://reviews.llvm.org/D36635 Files: tools/clang-fuzzer/Dockerfile tools/clang-fuzzer/README.txt Index: tools/clang-fuzzer/README.txt ===

[PATCH] D36635: Add a Dockerfile for clang-proto-fuzzer

kcc added inline comments. Comment at: tools/clang-fuzzer/Dockerfile:22 +# Get LLVM +RUN svn co http://llvm.org/svn/llvm-project/llvm/trunk llvm +RUN cd llvm/tools && svn co http://llvm.org/svn/llvm-project/cfe/trunk clang -r $(cd ../ && svn info | grep Revision | awk '{print $2

[PATCH] D29660: [OpenMP] Add flag for overwriting default PTX version for OpenMP targets

gtbercea added a comment. In https://reviews.llvm.org/D29660#839550, @alekseyshl wrote: > http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/7109 > failed on r310718, please fix. Couldn't fix/find the actual error so for now, just moving the flag patch tests to openmp-offloa

[PATCH] D36580: [OpenCL] Support variable memory scope in atomic builtins

rjmccall added inline comments. Comment at: include/clang/Basic/SyncScope.h:46 + Scopes.push_back(SyncScope::OpenCLSubGroup); + return Scopes; +} t-tye wrote: > Should there be an assert/static_assert in case SyncScope enum grows due to > other languages? It m

[PATCH] D36555: Move x86-specific sources to x86-specific source lists.

aheejin added a comment. Looks OK to me, but I haven't worked on this part of code either, so I think someone else should approve it. https://reviews.llvm.org/D36555 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-

[PATCH] D36635: Add a Dockerfile for clang-proto-fuzzer

hintonda added a comment. You may want to add an ARG statement to force docker to rerun the svn commands each time you invoke `docker build`. Otherwise it will reuse the cache since docker only looks at the RUN text, not it's result. To get around that, you can add an `ARG REVISION` declaratio

[PATCH] D29660: [OpenMP] Add flag for overwriting default PTX version for OpenMP targets

alekseyshl added a comment. In https://reviews.llvm.org/D29660#839550, @alekseyshl wrote: > http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/7109 > failed on r310718, please fix. In https://reviews.llvm.org/D29660#839647, @gtbercea wrote: > In https://reviews.llvm.org/D2

[PATCH] D36580: [OpenCL] Support variable memory scope in atomic builtins

t-tye accepted this revision. t-tye added a comment. This revision is now accepted and ready to land. LGTM other than suggested documentation and static_assert/unreachable comments. Comment at: lib/CodeGen/CGAtomic.cpp:696 +if (S != Default) + SI->addCase(Builder.getIn

r310772 - Disabling openmp-offload.c on linux until it is stabilized on all local configurations.

Author: alekseyshl Date: Fri Aug 11 16:10:39 2017 New Revision: 310772 URL: http://llvm.org/viewvc/llvm-project?rev=310772&view=rev Log: Disabling openmp-offload.c on linux until it is stabilized on all local configurations. Differential revision: https://reviews.llvm.org/D29660 Modified: c

  1   2   >