[PATCH] D88630: [clang/CMake] Respect LLVM_TOOLS_INSTALL_DIR

mgorny added a comment. Well, I've tested it and it doesn't seem to break anything for us (Gentoo) but I don't really understand why you'd have a different install dir for LLVM and Clang. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88630/new/ h

[PATCH] D88634: [clangd] Extend the rename API.

hokein created this revision. hokein added a reviewer: sammccall. Herald added subscribers: usaxena95, kadircet, arphaman. Herald added a project: clang. hokein requested review of this revision. Herald added subscribers: MaskRay, ilya-biryukov. several changes: - return a structure result in ren

[PATCH] D86743: [analyzer] Ignore VLASizeChecker case that could cause crash

balazske added a comment. I do not know if it got overlooked but there is another patch D81061 that removes the crash. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86743/new/ https://reviews.llvm.org/D86743

[PATCH] D88590: [clangd] Add bencmark for measuring latency of DecisionForest model.

usaxena95 updated this revision to Diff 295478. usaxena95 added a comment. Added benchmark. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88590/new/ https://reviews.llvm.org/D88590 Files: clang-tools-extra/clangd/benchmarks/CMakeLists.txt clan

[PATCH] D88590: [clangd] Add bencmark for measuring latency of DecisionForest model.

usaxena95 updated this revision to Diff 295479. usaxena95 added a comment. Distribution change and better formatting. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88590/new/ https://reviews.llvm.org/D88590 Files: clang-tools-extra/clangd/benchm

[PATCH] D87962: [clang] Change the multi-character character constants from extension to implementation-defined.

nomanous added a comment. In D87962#2298021 , @aaron.ballman wrote: > Multichar literals are implementation-defined in C and conditionally > supported with implementation-defined semantics in C++. I agree that it may > make sense to warn about their use

[PATCH] D86743: [analyzer] Ignore VLASizeChecker case that could cause crash

NoQ added a comment. But that's for a different bug, no? Even if that patch somehow dodges *this* crash, the underlying problem of us being unable to account for mutability of VLA sizes still remains to be addressed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://re

[PATCH] D88338: [clangd] clangd --check: standalone diagnosis of common problems

kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land. thanks LGTM! Comment at: clang-tools-extra/clangd/tool/Check.cpp:107 + +Cmd = CDB->getFallbackCommand(File); +if (auto TrueCmd = CDB->getCompileCommand(File)) { -

[PATCH] D85528: [analyzer] Fix cast evaluation on scoped enums in ExprEngine

steakhal added a comment. ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85528/new/ https://reviews.llvm.org/D85528 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-

[PATCH] D88567: [clangd] Fix invalid UTF8 when extracting doc comments.

kadircet added inline comments. Comment at: clang-tools-extra/clangd/CodeCompletionStrings.cpp:93 + // Clang requires source to be UTF-8, but doesn't enforce this in comments. + if (!llvm::json::isUTF8(Doc)) +Doc = llvm::json::fixUTF8(Doc); sammccall wrote:

[PATCH] D88640: [Format] Don't treat compound extension headers (foo.proto.h) as foo.cc main file.

hokein created this revision. hokein added a reviewer: sammccall. Herald added a project: clang. hokein requested review of this revision. We receive internal bugs about this false positives after D86597 . Repository: rG LLVM Github Monorepo https://reviews.l

[PATCH] D88643: [NFC] Correct name of profile function to Profile in APValue

Tyker created this revision. Tyker added a reviewer: rsmith. Herald added a project: clang. Herald added a subscriber: cfe-commits. Tyker requested review of this revision. Capitalize the profile function of APValue such that it can be used by FoldingSetNodeID Repository: rG LLVM Github Monor

[PATCH] D88645: [Annotation] Allows annotation to carry some additional constant arguments.

Tyker created this revision. Tyker added a reviewer: aaron.ballman. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. Tyker requested review of this revision. Herald added a subscriber: jdoerfert. This allows using annotation in a much more contexts than it c

Re: [PATCH] D68364: Implement C++20's P0784 (More constexpr containers)

Hi Louis, ARM and AArch64 libcxx bots are still broken after this commit (host compiler is clang-8), logs are available here: http://lab.llvm.org:8011/builders/libcxx-libcxxabi-libunwind-aarch64-linux/builds/3143 Cheers, Yvan On Mon, 28 Sep 2020 at 23:44, Louis Dionne via Phabricator via cfe-com

[PATCH] D88648: Refactor fixed point conversion test.

ebevhan created this revision. Herald added subscribers: cfe-commits, bjope. Herald added a project: clang. ebevhan requested review of this revision. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D88648 Files: clang/test/Frontend/fixed_point_conversions.c clang/test/Fronten

[PATCH] D83144: Allow to specify macro names for android-comparison-in-temp-failure-retry.

fmayer added a comment. I do not have commit access. Could someone submit this patch please? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83144/new/ https://reviews.llvm.org/D83144 ___ cfe-commits mailing list cfe-commits@lists.llvm.org htt

[PATCH] D87449: [clang-tidy] Add new check for SEI CERT rule SIG30-C

balazske updated this revision to Diff 295538. balazske added a comment. Added support for C++ code. Improved detection of 'signal' function. Simplified collection of called functions. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87449/new/ https:

[PATCH] D88640: [Format] Don't treat compound extension headers (foo.proto.h) as foo.cc main file.

sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang/lib/Tooling/Inclusions/HeaderIncludes.cpp:237 + + StringRef HeaderStem = + llvm::sys::path::stem(IncludeName.drop_front(1).drop_back( --

[PATCH] D87449: [clang-tidy] Add new check for SEI CERT rule SIG30-C

balazske marked 12 inline comments as done. balazske added inline comments. Comment at: clang-tools-extra/clang-tidy/cert/SignalHandlerCheck.cpp:95 + std::deque> CalledFunctions{ + {HandlerDecl, HandlerExpr}}; + baloghadamsoftware wrote: > Do we really need

[PATCH] D77062: [analyzer] Improve zero assumption in CStringChecke::assumeZero

ASDenysPetrov added a comment. @steakhal > @ASDenysPetrov Do you still want to rework the API of the `assumeZero`? This patch more looks like NFC, being just refactored. Actually I see that if we find and fix the root cause, we can freely refuse this patch. Another thing I see is that this patc

[PATCH] D88613: [flang] Semantic analysis for FINAL subroutines

klausler updated this revision to Diff 295423. klausler added a comment. Previous update to this review had inadvertent changes to other files because I neglected to rebase after updating master; now fixed. Sorry for the scare! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D88634: [clangd] Extend the rename API.

sammccall added inline comments. Comment at: clang-tools-extra/clangd/ClangdServer.cpp:408 return CB(InpAST.takeError()); -auto &AST = InpAST->AST; -const auto &SM = AST.getSourceManager(); -auto Loc = sourceLocationInMainFile(SM, Pos); -if (!Loc) - re

[PATCH] D88314: Added llvm-string-referencing check

bogser01 updated this revision to Diff 295553. bogser01 added a comment. Fixed failing unit test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88314/new/ https://reviews.llvm.org/D88314 Files: clang-tools-extra/clang-tidy/llvm/CMakeLists.txt c

[clang-tools-extra] 79fbcbf - [clangd] clangd --check: standalone diagnosis of common problems

Author: Sam McCall Date: 2020-10-01T15:47:47+02:00 New Revision: 79fbcbff41734e3d07e6200d33c3e40732dfae6a URL: https://github.com/llvm/llvm-project/commit/79fbcbff41734e3d07e6200d33c3e40732dfae6a DIFF: https://github.com/llvm/llvm-project/commit/79fbcbff41734e3d07e6200d33c3e40732dfae6a.diff LO

[PATCH] D88338: [clangd] clangd --check: standalone diagnosis of common problems

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. sammccall marked an inline comment as done. Closed by commit rG79fbcbff4173: [clangd] clangd --check: standalone diagnosis of common problems (authored by sammccall).

[PATCH] D87962: [clang] Change the multi-character character constants from extension to implementation-defined.

nomanous updated this revision to Diff 295559. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87962/new/ https://reviews.llvm.org/D87962 Files: clang/include/clang/Basic/DiagnosticLexKinds.td clang/test/CXX/lex/lex.literal/lex.ccon/p1.cpp clang/test/FixIt/format.m clang/test/Lexer

[clang] 5011d43 - Migrate Declarators to use the List API

Author: Eduardo Caldas Date: 2020-10-01T13:56:31Z New Revision: 5011d43108d1de30a056d66e73fa19062e0e84b7 URL: https://github.com/llvm/llvm-project/commit/5011d43108d1de30a056d66e73fa19062e0e84b7 DIFF: https://github.com/llvm/llvm-project/commit/5011d43108d1de30a056d66e73fa19062e0e84b7.diff LOG

[PATCH] D88566: be more specific when testing for no fuse-ld warnings

dmgreen added a comment. In the past we have usually disabled the downstream warning for similar catch-all warning lines. Comment at: clang/test/Driver/fuse-ld.c:5 // RUN: FileCheck %s --check-prefix=CHECK-ABSOLUTE-LD // CHECK-ABSOLUTE-LD: warning: '-fuse-ld=' taking a pat

[PATCH] D88403: Migrate Declarators to use the List API

This revision was automatically updated to reflect the committed changes. Closed by commit rG5011d43108d1: Migrate Declarators to use the List API (authored by eduucaldas). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88403/new/ https://reviews.llv

[PATCH] D88338: [clangd] clangd --check: standalone diagnosis of common problems

thakis added a comment. Might break tests: http://45.33.8.238/linux/29238/step_9.txt Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88338/new/ https://reviews.llvm.org/D88338 ___ cfe-commits mailing list

[PATCH] D88566: be more specific when testing for no fuse-ld warnings

dmgreen added a comment. (Not that I'm against this, either way sounds fine to me) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88566/new/ https://reviews.llvm.org/D88566 ___ cfe-commits mailing list cf

[clang-tools-extra] 30d07b1 - Revert "[clangd] clangd --check: standalone diagnosis of common problems"

Author: Sam McCall Date: 2020-10-01T16:10:03+02:00 New Revision: 30d07b14a274f075a01d201ad59723ca1a4a9b57 URL: https://github.com/llvm/llvm-project/commit/30d07b14a274f075a01d201ad59723ca1a4a9b57 DIFF: https://github.com/llvm/llvm-project/commit/30d07b14a274f075a01d201ad59723ca1a4a9b57.diff LO

[PATCH] D88275: [ASTMatchers] Add matcher `hasParentIgnoringImplicit`.

aaron.ballman added a comment. In D88275#2303283 , @ymandel wrote: >> I'm not concerned about the basic idea behind the proposed matcher, I'm only >> worried we're making AST matching more confusing by having two different >> ways of inconsistently accom

[PATCH] D88338: [clangd] clangd --check: standalone diagnosis of common problems

sammccall added a comment. Thanks, I think this fails whenever `compile_commands.json` doesn't exist in the tree or under `build`. (Which of course it does in my local checkout...). Reverted, probably need to rename/copy the test file. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[clang-tools-extra] f6b1323 - Reland [clangd] clangd --check: standalone diagnosis of common problems

Author: Sam McCall Date: 2020-10-01T16:18:18+02:00 New Revision: f6b1323bc680812e04904293854c356530985bcd URL: https://github.com/llvm/llvm-project/commit/f6b1323bc680812e04904293854c356530985bcd DIFF: https://github.com/llvm/llvm-project/commit/f6b1323bc680812e04904293854c356530985bcd.diff LO

[PATCH] D54222: [clang-tidy] Add a check to detect returning static locals in public headers

jranieri-grammatech updated this revision to Diff 295563. jranieri-grammatech added a comment. Herald added a project: clang. In D54222#2295374 , @JonasToth wrote: > Do you have data for other projects? As this is not a very common thing and > probably di

[PATCH] D87962: [clang] Change the multi-character character constants from extension to implementation-defined.

efriedma added a comment. > Or should we change the four character case to "Remark" so it wouldn't be > warned even with the "-pandemic" option? There seems no other choice. There's a DefaultIgnore modifier for warnings that turns then off by default. We use this sparingly, since it's hard to

[PATCH] D87962: [clang] Change the multi-character character constants from extension to implementation-defined.

aaron.ballman requested changes to this revision. aaron.ballman added a comment. This revision now requires changes to proceed. In D87962#2305375 , @nomanous wrote: > In D87962#2298021 , @aaron.ballman > wrote: > >

[PATCH] D88524: [CUDA][HIP] Fix bound arch for offload action for fat binary

yaxunl updated this revision to Diff 295572. yaxunl added a comment. Herald added a subscriber: jholewinski. add CudaArch::UNUSED as suggested by Artem. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88524/new/ https://reviews.llvm.org/D88524 Files: clang/include/clang/Basic/Cuda.h

[PATCH] D88491: [ASTContext] Use AllowCXX in all merge*Type methods, strip references

aaron.ballman added a comment. Do you have test cases for this change? I didn't see any relevant ones in D88384 . Comment at: clang/lib/AST/ASTContext.cpp:9174 bool ASTContext::typesAreBlockPointerCompatible(QualType LHS, QualType RHS) { re

[PATCH] D88659: [FE]Split SuitableAlign into two parts

Xiangling_L created this revision. Xiangling_L added reviewers: hubert.reinterpretcast, zarko, Jason, jyknight, efriedma. Herald added subscribers: cfe-commits, luismarques, apazos, sameer.abuasal, pzheng, s.egerton, lenary, Jim, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, atanasy

[PATCH] D88424: [clang][codegen] Remove the insertion of `correctly-rounded-divide-sqrt-fp-math` fn-attr.

This revision was automatically updated to reflect the committed changes. Closed by commit rG8c36eaf03772: [clang][opencl][codegen] Remove the insertion of `correctly-rounded-divide-sqrt… (authored by hliao). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.

[clang] 8c36eaf - [clang][opencl][codegen] Remove the insertion of `correctly-rounded-divide-sqrt-fp-math` fn-attr.

Author: Michael Liao Date: 2020-10-01T11:07:39-04:00 New Revision: 8c36eaf0377285acb89c319582d9666e60f42007 URL: https://github.com/llvm/llvm-project/commit/8c36eaf0377285acb89c319582d9666e60f42007 DIFF: https://github.com/llvm/llvm-project/commit/8c36eaf0377285acb89c319582d9666e60f42007.diff

[PATCH] D88314: Added llvm-string-referencing check

bogser01 updated this revision to Diff 295580. bogser01 added a comment. Fixed unit test 2 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88314/new/ https://reviews.llvm.org/D88314 Files: clang-tools-extra/clang-tidy/add_new_check.py clang-tool

[PATCH] D83088: Introduce CfgTraits abstraction

nhaehnle added a comment. Herald added a subscriber: tatianashp. ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83088/new/ https://reviews.llvm.org/D83088 ___ cfe-commits mailing list cfe-commits@lis

[PATCH] D88446: docs: add documentation describing API Notes

compnerd marked 3 inline comments as done. compnerd added inline comments. Comment at: clang/docs/APINotes.rst:45 +Clang will search for API notes files next to module maps only when passed the +``-fapinotes-modules`` option. + rsmith wrote: > Can we add a hyphen

[PATCH] D88664: [AST] do not error on APFloat invalidOp in default mode

spatel created this revision. spatel added reviewers: rjmccall, efriedma, sepavloff. Herald added a subscriber: mcrosier. spatel requested review of this revision. If FP exceptions are ignored, we should not error out of compilation just because APFloat indicated an exception. This is required a

[PATCH] D88665: [ASTImporter][AST] Fix structural equivalency crash on dependent FieldDecl

martong created this revision. martong added reviewers: balazske, teemperor, shafik, a_sidorin. Herald added subscribers: cfe-commits, gamesh411, Szelethus, dkrupp, rnkovacs. Herald added a reviewer: a.sidorin. Herald added a project: clang. martong requested review of this revision. Repository:

[PATCH] D88665: [ASTImporter][AST] Fix structural equivalency crash on dependent FieldDecl

martong added a comment. The added test causes the below assertion in the baseline (w/o the fix): ASTTests: ../../git/llvm-project/clang/lib/AST/ExprConstant.cpp:14543: llvm::APSInt clang::Expr::EvaluateKnownConstInt(const clang::ASTContext&, llvm::SmallVectorImpl >*) const: Assertion `!isVal

[PATCH] D88666: DirectoryWatcher: add an implementation for Windows

compnerd created this revision. compnerd added a reviewer: amccarth. Herald added a project: clang. compnerd requested review of this revision. This implements the directory watcher on Windows. It does the most naive thing for simplicity. ReadDirectoryChangesW is used to monitor the changes. Ho

[PATCH] D88590: [clangd] Add bencmark for measuring latency of DecisionForest model.

adamcz added a comment. Would it make sense to set random seed to something fixed to make this more reproducible? Comment at: clang-tools-extra/clangd/benchmarks/CompletionModel/DecisionForestBenchmark.cpp:2 +//===--- DecisionForestBenchmark.cpp - Benchmark for code completio

[PATCH] D88664: [AST] do not error on APFloat invalidOp in default mode

rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. LGTM. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88664/new/ https://reviews.llvm.org/D88664 ___ cfe-commits mailing list cfe-commit

[clang-tools-extra] 45698ac - [clangd] Split DecisionForest Evaluate() into one func per tree.

Author: Utkarsh Saxena Date: 2020-10-01T18:07:23+02:00 New Revision: 45698ac0052ae5b1c5beb739636396a5b7263966 URL: https://github.com/llvm/llvm-project/commit/45698ac0052ae5b1c5beb739636396a5b7263966 DIFF: https://github.com/llvm/llvm-project/commit/45698ac0052ae5b1c5beb739636396a5b7263966.diff

[PATCH] D88536: [clangd] Split DecisionForest Evaluate() into one func per tree.

This revision was automatically updated to reflect the committed changes. Closed by commit rG45698ac0052a: [clangd] Split DecisionForest Evaluate() into one func per tree. (authored by usaxena95). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88536/n

[PATCH] D88668: [CUDA] Add support for 11.1

kiwixz created this revision. kiwixz added reviewers: jlebar, tra. kiwixz added a project: clang. Herald added subscribers: cfe-commits, yaxunl, jholewinski. kiwixz requested review of this revision. It looks like 11.1 doesn't have a version.txt file, so I changed the default guess in this case f

[PATCH] D88668: [CUDA] Add support for 11.1

jlebar added a comment. > It looks like 11.1 doesn't have a version.txt file Yikes, this is a problem if we can't tell the difference between CUDA versions! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88668/new/ https://reviews.llvm.org/D88668

[PATCH] D88659: [FE]Split SuitableAlign into two parts

jyknight added a comment. Hm, to start with, the current state of this confuses me. In GCC, the preprocessor macro `__BIGGEST_ALIGNMENT__` was supposed to expose the alignment used by `__attribute__((aligned))` (no arg specified), as well the alignment used for alloca. However, this is no longe

[PATCH] D88668: [CUDA] Add support for 11.1

kiwixz added a comment. In D88668#2306393 , @jlebar wrote: >> It looks like 11.1 doesn't have a version.txt file > > Yikes, this is a problem if we can't tell the difference between CUDA > versions! It looks like a blunder from NVIDIA, CUDA 11.1 actually

[PATCH] D88590: [clangd] Add bencmark for measuring latency of DecisionForest model.

adamcz added a comment. Also typo in commit description: bencmark Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88590/new/ https://reviews.llvm.org/D88590 ___ cfe-commits mailing list cfe-commits@lists.l

[PATCH] D88666: DirectoryWatcher: add an implementation for Windows

plotfi added a comment. Overall looks good. Comment at: clang/lib/DirectoryWatcher/windows/DirectoryWatcher-windows.cpp:95 + CreateEvent(NULL, /*bManualReset=*/TRUE, /*bInitialState=*/FALSE, NULL); + assert(ovlIO.hEvent); + Can this be an assert with some

[PATCH] D88668: [CUDA] Add support for 11.1

tra added a comment. Generally speaking the default should be conservative. It does us no good if we generate PTX 99.99, but discover that ptxas does not support it. Granted, these days 7.0 is also the wrong default as it's pretty ancient. IMO bumping it to 9.0 and GPU arch to sm_30 would be se

[PATCH] D88666: DirectoryWatcher: add an implementation for Windows

aaron.ballman added a comment. I wonder if we should unit test this functionality by having some tests that create and remove files that are watched. I'm not 100% convinced that is a great idea, but not having test coverage for the change is also not really a great idea either. Thoughts welcome

[PATCH] D88666: DirectoryWatcher: add an implementation for Windows

compnerd added inline comments. Comment at: clang/lib/DirectoryWatcher/windows/DirectoryWatcher-windows.cpp:95 + CreateEvent(NULL, /*bManualReset=*/TRUE, /*bInitialState=*/FALSE, NULL); + assert(ovlIO.hEvent); + plotfi wrote: > Can this be an assert with so

[PATCH] D88666: DirectoryWatcher: add an implementation for Windows

compnerd added inline comments. Comment at: clang/lib/DirectoryWatcher/windows/DirectoryWatcher-windows.cpp:87 +DWORD dwLength = GetFinalPathNameByHandleW(hDirectory, NULL, 0, 0); +std::unique_ptr buffer{new WCHAR[dwLength + 1]}; +(void)GetFinalPathNameByHandleW(hDire

[PATCH] D88566: be more specific when testing for no fuse-ld warnings

MaskRay added inline comments. Comment at: clang/test/Driver/fuse-ld.c:15 // RUN: FileCheck %s --check-prefix=CHECK-NO-WARN -// CHECK-NO-WARN-NOT: warning: +// CHECK-NO-WARN-NOT: warning: 'fuse-ld' How does this line trigger unrelated warnings? Can you dump

[PATCH] D88666: DirectoryWatcher: add an implementation for Windows

compnerd added a comment. @aaron.ballman - I completely agree with you about the testing. The interfaces are tested via https://github.com/llvm/llvm-project/blob/master/clang/unittests/DirectoryWatcher/CMakeLists.txt, which now that I look at again, seems to need an additional case for system

[clang-tools-extra] 9d40fb8 - Allow to specify macro names for android-comparison-in-temp-failure-retry

Author: Florian Mayer Date: 2020-10-01T10:09:26-07:00 New Revision: 9d40fb808fd0fbd33eb3b50c20d7f402de5db91e URL: https://github.com/llvm/llvm-project/commit/9d40fb808fd0fbd33eb3b50c20d7f402de5db91e DIFF: https://github.com/llvm/llvm-project/commit/9d40fb808fd0fbd33eb3b50c20d7f402de5db91e.diff

[PATCH] D83144: Allow to specify macro names for android-comparison-in-temp-failure-retry.

This revision was automatically updated to reflect the committed changes. Closed by commit rG9d40fb808fd0: Allow to specify macro names for android-comparison-in-temp-failure-retry (authored by fmayer, committed by george.burgess.iv). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACT

[PATCH] D88278: [PowerPC] Add builtins for xvtdiv(dp|sp) and xvtsqrt(dp|sp).

amyk accepted this revision as: amyk. amyk added a comment. This revision is now accepted and ready to land. Overall I think this LGTM. Please correct me if I am wrong but I think the description of the functions need to be updated to: int vec_test_swdiv(vector double v1, vector double v2);

[PATCH] D88524: [CUDA][HIP] Fix bound arch for offload action for fat binary

tra accepted this revision. tra added inline comments. This revision is now accepted and ready to land. Comment at: clang/lib/Basic/Cuda.cpp:95 const char *CudaArchToString(CudaArch A) { + if (A == CudaArch::UNUSED) +return ""; You could add a `{CudaArch::U

[PATCH] D88680: Add ability to turn off -fpch-instantiate-templates in clang-cl

shivanshu3 created this revision. shivanshu3 added reviewers: llunak, rnk, rsmith, hans, zahen. shivanshu3 added a project: clang. Herald added subscribers: cfe-commits, dang. shivanshu3 requested review of this revision. A lot of our code building with clang-cl.exe using Clang 11 was failing with

[PATCH] D88664: [AST] do not error on APFloat invalidOp in default mode

This revision was automatically updated to reflect the committed changes. Closed by commit rG686eb0d8ded9: [AST] do not error on APFloat invalidOp in default mode (authored by spatel). Herald added a project: clang. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://review

[clang] 686eb0d - [AST] do not error on APFloat invalidOp in default mode

Author: Sanjay Patel Date: 2020-10-01T13:46:45-04:00 New Revision: 686eb0d8ded9159b090c3ef7b33a422e1f05166e URL: https://github.com/llvm/llvm-project/commit/686eb0d8ded9159b090c3ef7b33a422e1f05166e DIFF: https://github.com/llvm/llvm-project/commit/686eb0d8ded9159b090c3ef7b33a422e1f05166e.diff

[PATCH] D88680: Add ability to turn off -fpch-instantiate-templates in clang-cl

shivanshu3 added a comment. Note that I do not have commit access and this change will have to be committed by someone else on my behalf. Please use the following commit author details: "Shivan Goyal " Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D

[PATCH] D63640: [clang] Improve Serialization/Imporing of APValues

Tyker updated this revision to Diff 295626. Tyker marked 15 inline comments as done. Tyker added a comment. addressed comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63640/new/ https://reviews.llvm.org/D63640 Files: clang/include/clang/AS

[PATCH] D63640: [clang] Improve Serialization/Imporing of APValues

Tyker added inline comments. Comment at: clang/lib/AST/ASTImporter.cpp:9010 + ToPath[Idx] = + cast(const_cast(ImpDecl.get())); +} rsmith wrote: > We want the path in an `APValue` to be canonical, but importing a canonical > decl might result in

[clang] c1b209c - [Format] Don't treat compound extension headers (foo.proto.h) as foo.cc main-file header.

Author: Haojian Wu Date: 2020-10-01T19:57:57+02:00 New Revision: c1b209cc61290f1ce1243470b825e0994645cb7d URL: https://github.com/llvm/llvm-project/commit/c1b209cc61290f1ce1243470b825e0994645cb7d DIFF: https://github.com/llvm/llvm-project/commit/c1b209cc61290f1ce1243470b825e0994645cb7d.diff LO

[PATCH] D88477: [analyzer] Overwrite cast type in getBinding only if that was null originally

steakhal added a comment. In D88477#2304708 , @NoQ wrote: > I'm trying to say that the value produced by the load should not be the same > as the stored value, because these two values are of different types. When > exactly does the first value change in

[PATCH] D77062: [analyzer] Improve zero assumption in CStringChecke::assumeZero

steakhal added a comment. In D77062#2305856 , @ASDenysPetrov wrote: > @steakhal > >> @ASDenysPetrov Do you still want to rework the API of the `assumeZero`? > > This patch more looks like NFC, being just refactored. Fine. > Actually I see that if we fin

[PATCH] D88345: [CUDA] Allow local `static const {__constant__, __device__}` variables.

tra updated this revision to Diff 295635. tra retitled this revision from "[CUDA] Allow local `static {__constant__, __device__}` variables." to "[CUDA] Allow local `static const {__constant__, __device__}` variables.". tra edited the summary of this revision. tra added a comment. Herald added a

[PATCH] D88345: [CUDA] Allow local `static const {__constant__, __device__}` variables.

tra updated this revision to Diff 295636. tra added a comment. Herald added a reviewer: jdoerfert. Fixed a test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88345/new/ https://reviews.llvm.org/D88345 Files: clang/include/clang/Basic/Diagnostic

[clang] 149f5b5 - [APFloat] convert SNaN to QNaN in convert() and raise Invalid signal

Author: Sanjay Patel Date: 2020-10-01T14:37:38-04:00 New Revision: 149f5b573c79eac0c519ada4d2f7c50e17796cdf URL: https://github.com/llvm/llvm-project/commit/149f5b573c79eac0c519ada4d2f7c50e17796cdf DIFF: https://github.com/llvm/llvm-project/commit/149f5b573c79eac0c519ada4d2f7c50e17796cdf.diff

[PATCH] D88238: [APFloat] convert SNaN to QNaN in convert() and raise Invalid signal

This revision was automatically updated to reflect the committed changes. Closed by commit rG149f5b573c79: [APFloat] convert SNaN to QNaN in convert() and raise Invalid signal (authored by spatel). Herald added subscribers: cfe-commits, jrtc27. Herald added a project: clang. Repository: rG LLVM

[PATCH] D87449: [clang-tidy] Add new check for SEI CERT rule SIG30-C

aaron.ballman added inline comments. Comment at: clang-tools-extra/clang-tidy/cert/SignalHandlerCheck.cpp:33 + + for (const FunctionDecl *D : Node.redecls()) +if (D->getASTContext().getSourceManager().isInSystemHeader( I'm not certain I understand why we're

[PATCH] D88694: [CMake][Fuchsia] Don't set WIN32 API, rely on autodetection

phosek created this revision. phosek added a reviewer: leonardchan. Herald added subscribers: cfe-commits, mgorny. Herald added a project: clang. phosek requested review of this revision. We prefer autodetection here to avoid persisting this configuration in the generated __config header which is

[PATCH] D87989: [Flang][Driver] Add infrastructure for basic frontend actions and file I/O

awarzynski added a comment. @CarolineConcatto thank you again for working on this! The structure is good, but IMHO this patch could be polished a bit more. Overall: - could you make sure that this patch does not change the output from `clang -help`? - doxygen comments are consistent - unittests

[PATCH] D88694: [CMake][Fuchsia] Don't set WIN32 API, rely on autodetection

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGde47e7122f69: [CMake][Fuchsia] Don't set WIN32 API, rely on autodetection (authored by phosek). Repository: rG LLVM Github Monorepo CHANGES SINCE

[clang] de47e71 - [CMake][Fuchsia] Don't set WIN32 API, rely on autodetection

Author: Petr Hosek Date: 2020-10-01T12:35:52-07:00 New Revision: de47e7122f69d56399c4f8864ba279e5ce635970 URL: https://github.com/llvm/llvm-project/commit/de47e7122f69d56399c4f8864ba279e5ce635970 DIFF: https://github.com/llvm/llvm-project/commit/de47e7122f69d56399c4f8864ba279e5ce635970.diff LO

[PATCH] D88345: [CUDA] Allow local `static const {__constant__, __device__}` variables.

yaxunl added inline comments. Comment at: clang/test/SemaCUDA/device-var-init.cu:404 __host__ __device__ void hd_sema() { static int x = 42; } how does this work in device compilation? Is this equivalent to `static __device__ int x = 42`? Repository: rG

[clang] 35ecc7f - [clang][Sema] Fix PR47676: Handle dependent AltiVec C-style cast

Author: Hubert Tong Date: 2020-10-01T15:57:01-04:00 New Revision: 35ecc7fe49ba881a77e8146b51870a60a52b211f URL: https://github.com/llvm/llvm-project/commit/35ecc7fe49ba881a77e8146b51870a60a52b211f DIFF: https://github.com/llvm/llvm-project/commit/35ecc7fe49ba881a77e8146b51870a60a52b211f.diff L

[PATCH] D88526: [clang][Sema] Fix PR47676: Handle dependent AltiVec C-style cast

This revision was automatically updated to reflect the committed changes. Closed by commit rG35ecc7fe49ba: [clang][Sema] Fix PR47676: Handle dependent AltiVec C-style cast (authored by hubert.reinterpretcast). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm

[PATCH] D88603: [WebAssembly] Add support for DWARF type units

aardappel added inline comments. Comment at: llvm/test/DebugInfo/WebAssembly/dwarf-headers.ll:47 +; +; SINGLE-4: .debug_types contents: +; SINGLE-4: 0x: Type Unit: {{.*}} version = 0x0004, abbr_offset I guess this doesn't actually test that only one copy

[PATCH] D88524: [CUDA][HIP] Fix bound arch for offload action for fat binary

yaxunl marked an inline comment as done. yaxunl added inline comments. Comment at: clang/lib/Basic/Cuda.cpp:95 const char *CudaArchToString(CudaArch A) { + if (A == CudaArch::UNUSED) +return ""; tra wrote: > You could add a `{CudaArch::UNUSED, "", ""}` to t

[libclc] 1c1a810 - libclc: Use find_package to find Python 3 and require it

Author: Aaron Puchert Date: 2020-10-01T22:31:33+02:00 New Revision: 1c1a8105580784c96212db1afc097a844740bc69 URL: https://github.com/llvm/llvm-project/commit/1c1a8105580784c96212db1afc097a844740bc69 DIFF: https://github.com/llvm/llvm-project/commit/1c1a8105580784c96212db1afc097a844740bc69.diff

[PATCH] D88345: [CUDA] Allow local `static const {__constant__, __device__}` variables.

tra added a comment. I've verified that clang with this patch can compile Tensorflow and that it can also compile `cooperative_groups.h` from CUDA-11. Comment at: clang/test/SemaCUDA/device-var-init.cu:404 __host__ __device__ void hd_sema() { static int x = 42; }

[PATCH] D88338: [clangd] clangd --check: standalone diagnosis of common problems

rsmith added inline comments. Comment at: clang-tools-extra/clangd/test/check.test:1 +# RUN: clangd -log=verbose -check 2>&1 | FileCheck -strict-whitespace %s + This test implicitly parses a source file that `#include`s standard library headers, and fails if tho

[PATCH] D88338: [clangd] clangd --check: standalone diagnosis of common problems

hoy added inline comments. Comment at: clang-tools-extra/clangd/tool/Check.cpp:243 + size_t N = 50; + auto xxx = std::string(N, 'x'); +)cpp"; Hi, I'm seeing an error with this "tweak: ExpandAutoType ==> FAIL: Could not deduce type for 'auto' type".

[PATCH] D74144: [OPENMP50]Add basic support for array-shaping operation.

cchen added a comment. Herald added subscribers: sstefan1, yaxunl. @ABataev, the below test is extracted from Sollve test suite and Clang now emit: test.c:17:35: error: subscripted value is not an array or pointer #pragma omp target update to( (([N][N])foo)[1:M] )

[PATCH] D74144: [OPENMP50]Add basic support for array-shaping operation.

ABataev added a comment. In D74144#2307454 , @cchen wrote: > @ABataev, the below test is extracted from Sollve test suite and Clang now > emit: > > test.c:17:35: error: subscripted value is not an array or pointer > #pragma omp target update to(

[PATCH] D88712: [CGBuiltin] Respect asm labels and redefine_extname for builtins with specialized emitting

MaskRay created this revision. MaskRay added reviewers: joerg, jyknight, rsmith. Herald added a project: clang. Herald added a subscriber: cfe-commits. MaskRay requested review of this revision. rL131311 added `asm()` support for builtin functions, but `asm()`

  1   2   >