[PATCH] D105426: [clangd] Include Sanity as a library: Find all references to symbols in the file

kbobyrev updated this revision to Diff 365369. kbobyrev added a comment. Rename. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105426/new/ https://reviews.llvm.org/D105426 Files: clang-tools-extra/clangd/CMakeLists.txt clang-tools-extra/clangd

[PATCH] D105177: [clangd] Implemented indexing of standard library

kuhnel updated this revision to Diff 365370. kuhnel added a comment. tried to fix Windows build failure Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105177/new/ https://reviews.llvm.org/D105177 Files: clang-tools-extra/clangd/CMakeLists.txt c

[PATCH] D100810: Use `GNUInstallDirs` to support custom installation dirs. -- LLVM

Ericson2314 added inline comments. Comment at: llvm/CMakeLists.txt:589 CACHE STRING "Doxygen-generated HTML documentation install directory") -set(LLVM_INSTALL_OCAMLDOC_HTML_DIR "share/doc/llvm/ocaml-html" +set(LLVM_INSTALL_OCAMLDOC_HTML_DIR "${CMAKE_INSTALL_DOCDIR}/${proje

[PATCH] D106876: Remove non-affecting module maps from PCM files.

ilyakuteev updated this revision to Diff 365381. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106876/new/ https://reviews.llvm.org/D106876 Files: clang/include/clang/Serialization/ASTWriter.h clang/lib/Serialization/ASTWriter.cpp Index: clang/

[PATCH] D106137: [flang][driver] Add support for Frontend Plugins

stuartellis updated this revision to Diff 365411. stuartellis marked an inline comment as done. stuartellis added a comment. Address review comments Small changes due to rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106137/new/ https://revie

[PATCH] D105177: [clangd] Implemented indexing of standard library

kuhnel updated this revision to Diff 365415. kuhnel added a comment. fixed a couple of bugs - wrong usage of llvm::unique - wrong usage of static pointer Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105177/new/ https://reviews.llvm.org/D105177 F

[PATCH] D97753: [clang-tidy] Add a check for enforcing minimum length for variable names

aaron.ballman added inline comments. Comment at: clang-tools-extra/clang-tidy/readability/ReadabilityTidyModule.cpp:136 +CheckFactories.registerCheck( +"readability-variable-length"); } 0x8000- wrote: > aaron.ballman wrote: > > 0x8000- wro

[PATCH] D107667: [clang/test] Run thinlto-clang-diagnostic-handler-in-be.c on x86

thopre updated this revision to Diff 365425. thopre added a comment. Use x86_64-linux-gnu as triplet Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107667/new/ https://reviews.llvm.org/D107667 Files: clang/test/CodeGen/thinlto-clang-diagnostic-ha

[PATCH] D107095: Implement #pragma clang header_unsafe

aaron.ballman added inline comments. Comment at: clang/test/Lexer/Inputs/unsafe-macro-2.h:23-26 +// not-expected-warning@+1{{macro 'UNSAFE_MACRO_2' has been marked as unsafe for use in headers}} +#undef UNSAFE_MACRO_2 +// not-expected-warning@+1{{macro 'UNSAFE_MACRO_2' has been

[PATCH] D107095: Implement #pragma clang header_unsafe

aaron.ballman added inline comments. Comment at: clang/include/clang/Basic/DiagnosticLexKinds.td:535 +def note_pp_macro_annotation : + Note<"macro marked %select{deprecated|header_unsafe}0 here">; + Repository: rG LLVM Github Monorepo CHANGES SINCE LAST AC

[clang] 5de6b1a - [OpenCL] Make pipes and workgroup optional for -fdeclare-opencl-builtins

Author: Sven van Haastregt Date: 2021-08-10T13:01:47+01:00 New Revision: 5de6b1acb5900be80515cf9fa253f8698fc57dca URL: https://github.com/llvm/llvm-project/commit/5de6b1acb5900be80515cf9fa253f8698fc57dca DIFF: https://github.com/llvm/llvm-project/commit/5de6b1acb5900be80515cf9fa253f8698fc57dca.

[PATCH] D106137: [flang][driver] Add support for Frontend Plugins

awarzynski accepted this revision. awarzynski added a comment. This revision is now accepted and ready to land. LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106137/new/ https://reviews.llvm.org/D106137 __

[PATCH] D105264: [X86] AVX512FP16 instructions enabling 2/6

LuoYuanke added inline comments. Comment at: clang/include/clang/Basic/BuiltinsX86.def:1860 +TARGET_BUILTIN(__builtin_ia32_minph512, "V32xV32xV32xIi", "ncV:512:", "avx512fp16") + +TARGET_BUILTIN(__builtin_ia32_minph256, "V16xV16xV16x", "ncV:256:", "avx512fp16,avx512vl

[PATCH] D107450: [clang-tidy] Fix wrong and missing warnings in performance-move-const-arg

Sockke updated this revision to Diff 365448. Sockke added a comment. update! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107450/new/ https://reviews.llvm.org/D107450 Files: clang-tools-extra/clang-tidy/performance/MoveConstArgCheck.cpp clang-tools-extra/clang-tidy/performance/Mov

[clang] bd63977 - [Parser] Fix attr infloop on "int x [[c"

Author: Sam McCall Date: 2021-08-10T15:03:08+02:00 New Revision: bd63977ca96acc7db2a546f56a5c0ed1fc93e22a URL: https://github.com/llvm/llvm-project/commit/bd63977ca96acc7db2a546f56a5c0ed1fc93e22a DIFF: https://github.com/llvm/llvm-project/commit/bd63977ca96acc7db2a546f56a5c0ed1fc93e22a.diff LO

[PATCH] D107693: [Parser] Fix attr infloop on "int x [[c"

This revision was automatically updated to reflect the committed changes. Closed by commit rGbd63977ca96a: [Parser] Fix attr infloop on "int x [[c" (authored by sammccall). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107693/new/ https://reviews.ll

[PATCH] D107450: [clang-tidy] Fix wrong and missing warnings in performance-move-const-arg

Sockke updated this revision to Diff 365455. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107450/new/ https://reviews.llvm.org/D107450 Files: clang-tools-extra/clang-tidy/performance/MoveConstArgCheck.cpp clang-tools-extra/clang-tidy/performance/MoveConstArgCheck.h clang-tools-ext

[PATCH] D107641: [clang-tidy] fix duplicate '{}' in cppcoreguidelines-pro-type-member-init

Sockke marked 2 inline comments as done. Sockke added a comment. Any thoughts? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107641/new/ https://reviews.llvm.org/D107641 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.ll

[PATCH] D69764: [clang-format] Add Left/Right Const fixer capability

MyDeveloperDay updated this revision to Diff 365456. MyDeveloperDay marked 2 inline comments as done. MyDeveloperDay added a comment. - Add support for both const and volatile alignment - change `ConstPlacement` to `CVQualifierAlignment` - add `CVQualifierOrder` to allow control over `const volati

[clang] 13a86c2 - [Sema] Preserve invalid CXXCtorInitializers using RecoveryExpr in initializer

Author: Sam McCall Date: 2021-08-10T15:16:52+02:00 New Revision: 13a86c2bb465edf315ecbac622d73d39abe7 URL: https://github.com/llvm/llvm-project/commit/13a86c2bb465edf315ecbac622d73d39abe7 DIFF: https://github.com/llvm/llvm-project/commit/13a86c2bb465edf315ecbac622d73d39abe7.diff LO

[PATCH] D101641: [Sema] Preserve invalid CXXCtorInitializers using RecoveryExpr in initializer

This revision was automatically updated to reflect the committed changes. Closed by commit rG13a86c2bb465: [Sema] Preserve invalid CXXCtorInitializers using RecoveryExpr in initializer (authored by sammccall). Changed prior to commit: https://reviews.llvm.org/D101641?vs=364846&id=365457#toc Re

[clang-tools-extra] 45abbaf - Revert "[clangd] Support `#pragma mark` in the outline"

Author: Florian Mayer Date: 2021-08-10T14:25:52+01:00 New Revision: 45abbaf2e5fbdf27c9f8ba01b34018b0be45b7c9 URL: https://github.com/llvm/llvm-project/commit/45abbaf2e5fbdf27c9f8ba01b34018b0be45b7c9 DIFF: https://github.com/llvm/llvm-project/commit/45abbaf2e5fbdf27c9f8ba01b34018b0be45b7c9.diff

[PATCH] D105904: [clangd] Support `#pragma mark` in the outline

fmayer reopened this revision. fmayer added a comment. This revision is now accepted and ready to land. This broke the UBsan buildbot: https://lab.llvm.org/buildbot/#/builders/85/builds/6103/steps/10/logs/stdio Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.l

[PATCH] D107450: [clang-tidy] Fix wrong and missing warnings in performance-move-const-arg

MTC added inline comments. Comment at: clang-tools-extra/test/clang-tidy/checkers/performance-move-const-arg.cpp:261 + showInt(std::move(a)); + // CHECK-MESSAGES: :[[@LINE-1]]:11: warning: std::move of the variable 'a' of the trivially-copyable type 'int' has no effect; consi

[PATCH] D106737: [clang] [hexagon] Add resource include dir

bcain updated this revision to Diff 365459. bcain added a comment. Suggestions from Fangrui: override the resource-dir with an explicit arg, disable for windows. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106737/new/ https://reviews.llvm.org/D1

[clang] 888876b - [clang] [hexagon] Add resource include dir

Author: Brian Cain Date: 2021-08-10T08:37:58-05:00 New Revision: 76ba272baf68ff38fcfc36c15ac2510bdea7 URL: https://github.com/llvm/llvm-project/commit/76ba272baf68ff38fcfc36c15ac2510bdea7 DIFF: https://github.com/llvm/llvm-project/commit/76ba272baf68ff38fcfc36c15ac2510bdea7.diff LO

[PATCH] D69764: [clang-format] Add Left/Right Const fixer capability

HazardyKnusperkeks added a comment. First off, I think it should be configured in a different way, to prepare the path for also formatting static, inline, etc. If this is kept there should be tests on what happens if there is const or volatile more than once in the string list, and when there a

[PATCH] D107703: [AST][clangd] Expose documentation of Attrs on hover.

sammccall updated this revision to Diff 365461. sammccall marked an inline comment as done. sammccall added a comment. Change assertion to defensive check Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107703/new/ https://reviews.llvm.org/D107703 F

[PATCH] D107703: [AST][clangd] Expose documentation of Attrs on hover.

sammccall added inline comments. Comment at: clang/utils/TableGen/ClangAttrEmitter.cpp:4220 + )cpp"; + std::vector Attrs = Records.getAllDerivedDefinitions("Attr"); + for (const auto *A : Attrs) { aaron.ballman wrote: > FWIW, this will miss `omp::sequence` and

[PATCH] D107095: Implement #pragma clang header_unsafe

beanz added inline comments. Comment at: clang/test/Lexer/Inputs/unsafe-macro-2.h:23-26 +// not-expected-warning@+1{{macro 'UNSAFE_MACRO_2' has been marked as unsafe for use in headers}} +#undef UNSAFE_MACRO_2 +// not-expected-warning@+1{{macro 'UNSAFE_MACRO_2' has been marked a

[PATCH] D97803: [clangd] Overload bundles are only deprecated if each overloads is.

sammccall added a comment. Herald added a project: clang-tools-extra. Whoops, lost this patch... In D97803#2602787 , @kbobyrev wrote: > So, if my understanding is correct, this will make the whole bundle > non-deprecated if at least one overload is not d

[PATCH] D107633: Set supported target for asan-use-callbacks test

thopre updated this revision to Diff 365465. thopre added a comment. Remove REQUIRES Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107633/new/ https://reviews.llvm.org/D107633 Files: clang/test/CodeGen/asan-use-callbacks.cpp Index: clang/test/

[PATCH] D107633: Set supported target for asan-use-callbacks test

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG1397e19129ef: Set supported target for asan-use-callbacks test (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTIO

[clang] 1397e19 - Set supported target for asan-use-callbacks test

Author: Thomas Preud'homme Date: 2021-08-10T15:01:44+01:00 New Revision: 1397e19129ef7aa6c2ba6f6018ff172a5022a1eb URL: https://github.com/llvm/llvm-project/commit/1397e19129ef7aa6c2ba6f6018ff172a5022a1eb DIFF: https://github.com/llvm/llvm-project/commit/1397e19129ef7aa6c2ba6f6018ff172a5022a1eb.

[PATCH] D107825: Define __HOS_AIX__ only for AIX target

Jake-Egan created this revision. Herald added subscribers: kbarton, nemanjai. Jake-Egan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D107825 Files: clang/lib/Basic/Target

[PATCH] D107450: [clang-tidy] Fix wrong and missing warnings in performance-move-const-arg

Quuxplusone added a comment. I'm not at all convinced that any of this complexity is worth it. Have you run this check on any large codebase to see how many false positives it has? Does it catch any true positives? Comment at: clang-tools-extra/test/clang-tidy/checkers/perfo

[PATCH] D69764: [clang-format] Add Left/Right Const fixer capability

MyDeveloperDay updated this revision to Diff 365472. MyDeveloperDay added a comment. - elide the braces - use references CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69764/new/ https://reviews.llvm.org/D69764 Files: clang/docs/ClangFormatStyleOptions.rst clang/docs/ReleaseNotes.rs

[PATCH] D107095: Implement #pragma clang header_unsafe

aaron.ballman added inline comments. Comment at: clang/test/Lexer/Inputs/unsafe-macro-2.h:23-26 +// not-expected-warning@+1{{macro 'UNSAFE_MACRO_2' has been marked as unsafe for use in headers}} +#undef UNSAFE_MACRO_2 +// not-expected-warning@+1{{macro 'UNSAFE_MACRO_2' has been

[PATCH] D107095: Implement #pragma clang header_unsafe

beanz added inline comments. Comment at: clang/test/Lexer/Inputs/unsafe-macro-2.h:23-26 +// not-expected-warning@+1{{macro 'UNSAFE_MACRO_2' has been marked as unsafe for use in headers}} +#undef UNSAFE_MACRO_2 +// not-expected-warning@+1{{macro 'UNSAFE_MACRO_2' has been marked a

[PATCH] D69764: [clang-format] Add Left/Right Const fixer capability

MyDeveloperDay added a comment. In D69764#2936827 , @HazardyKnusperkeks wrote: > First off, I think it should be configured in a different way, to prepare the > path for also formatting static, inline, etc. To be honest I think if we wanted to do that w

[PATCH] D69764: [clang-format] Add Left/Right Const fixer capability

MyDeveloperDay added a comment. > In D69764#2936827 , > @HazardyKnusperkeks wrote: > >> First off, I think it should be configured in a different way, to prepare >> the path for also formatting static, inline, etc. I'm wondering if I'm gravitating more a

[PATCH] D107095: Implement #pragma clang header_unsafe

aaron.ballman added inline comments. Comment at: clang/docs/LanguageExtensions.rst:3916 +processed by the preprocessor after the ``#pragma`` annotation will log a +a warning. For example: + May want to put something in here along the lines of: Redefining the macr

[PATCH] D107002: [PowerPC] Implement XL compatibility builtin __addex

lei updated this revision to Diff 365480. lei marked 2 inline comments as done. lei added a comment. update sema check condition and remove duplicate tc Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107002/new/ https://reviews.llvm.org/D107002 Fil

[PATCH] D107825: [AIX] Define __HOS_AIX__ macro only for AIX target

joerg accepted this revision. joerg added a comment. This revision is now accepted and ready to land. LGTM. Maybe include a small hint in the commit message that xlC never shipped cross-compiling support and the difference is therefore not observable anyway. Repository: rG LLVM Github Monorep

[PATCH] D107002: [PowerPC] Implement XL compatibility builtin __addex

lei added inline comments. Comment at: clang/test/CodeGen/builtins-ppc-xlcompat-pwr9-warning.c:4 +// RUN: -verify %s + +extern unsigned long long ull; NeHuang wrote: > can we also add the run lines for 64 bit LE Linux, 64 bit AIX and 32 bit AIX? > Will also n

[PATCH] D107242: [AIX] Define __HOS_AIX__ macro

cebowleratibm added a comment. > Given that we have a legacy XL macro with no legacy cross compiler I think > it's fine if we set this according to the target only. It's fully redundant > to `_AIX ` but we'll define it for any working use-case with the current xlC > compiler. If IBM has a com

[clang] 638dcea - [clang] Implement P0692R1 from C++20 (access checking on specializations and instantiations)

Author: Alex Orlov Date: 2021-08-10T19:20:50+04:00 New Revision: 638dcea010cfc280f428d0cc13f4aa8578a1d69d URL: https://github.com/llvm/llvm-project/commit/638dcea010cfc280f428d0cc13f4aa8578a1d69d DIFF: https://github.com/llvm/llvm-project/commit/638dcea010cfc280f428d0cc13f4aa8578a1d69d.diff LO

[PATCH] D92024: [clang] Implement P0692R1 from C++20 (access checking on specializations and instantiations)

This revision was automatically updated to reflect the committed changes. Closed by commit rG638dcea010cf: [clang] Implement P0692R1 from C++20 (access checking on specializations and… (authored by aorlov). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.or

[PATCH] D107095: Implement #pragma clang header_unsafe

beanz updated this revision to Diff 365487. beanz added a comment. Updated docs, added header-unsafe-macro diag group, and added test case to verify that the -Wpedantic-macros warnings don't trip on eachother. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.ll

[PATCH] D107696: [CodeComplete] Basic code completion for attribute names.

sammccall updated this revision to Diff 365490. sammccall added a comment. Herald added subscribers: sstefan1, krytarowski. Herald added a reviewer: jdoerfert. Add support and tests for underscore-guarding. Add tests that omp attributes are not supported. Repository: rG LLVM Github Monorepo C

[PATCH] D107703: [AST][clangd] Expose documentation of Attrs on hover.

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. This generally LGTM but the only other thing I'm wondering is whether there should be any clang-specific testing for this functionality. Right now, it relies on running the clang

[PATCH] D107717: [LLVM][CMake][NFC] Resolve FIXME: Rename LLVM_CMAKE_PATH to LLVM_CMAKE_DIR throughout the project

gAlfonso-bit added a comment. So does that mean this is ready to land? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107717/new/ https://reviews.llvm.org/D107717 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/c

[PATCH] D107836: [Attributes]: refactor to expose ParsedAttrInfo::acceptsLangOpts. NFC

sammccall created this revision. sammccall added a reviewer: aaron.ballman. sammccall requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. We will use this function to filter code completion of attributes. Repository: rG LLVM Github Monorepo

[PATCH] D107837: abseil-string-find-str-contains should not propose an edit for the three-parameter version of find().

tdl-g created this revision. tdl-g added a reviewer: ymandel. tdl-g requested review of this revision. Herald added a project: clang-tools-extra. Herald added a subscriber: cfe-commits. std::string, std::string_view, and absl::string_view all have a three-parameter version of find() which has a "

[PATCH] D107668: [OpenMP]Fix PR50336: Remove temporary files in the offload bundler tool

jdoerfert added a comment. Why the else? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107668/new/ https://reviews.llvm.org/D107668 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.l

[PATCH] D107668: [OpenMP]Fix PR50336: Remove temporary files in the offload bundler tool

jhuber6 added a comment. In D107668#2937230 , @jdoerfert wrote: > Why the else? If this input isn't an OffloadingAction then it is a `.o` file which we already list as a temporary file. If it was an OffloadingAction then we have an extra device-specifi

[PATCH] D107703: [AST][clangd] Expose documentation of Attrs on hover.

sammccall added a comment. In D107703#2937183 , @aaron.ballman wrote: > This generally LGTM but the only other thing I'm wondering is whether there > should be any clang-specific testing Makes sense. Here the "clang feature" is just the API, so I've ad

[PATCH] D107703: [AST][clangd] Expose documentation of Attrs on hover.

sammccall updated this revision to Diff 365499. sammccall added a comment. add unit test for getDocumentation Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107703/new/ https://reviews.llvm.org/D107703 Files: clang-tools-extra/clangd/Hover.cpp

[PATCH] D107775: [Clang][AST] Resolve FIXME: Remove ObjCObjectPointer from isSpecifierType

gAlfonso-bit updated this revision to Diff 365501. gAlfonso-bit added a comment. Rebased to main CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107775/new/ https://reviews.llvm.org/D107775 Files: clang/lib/AST/Type.cpp Index: clang/lib/AST/Type.cpp ==

[PATCH] D69764: [clang-format] Add Left/Right Const fixer capability

MyDeveloperDay updated this revision to Diff 365502. MyDeveloperDay added a comment. - Add CVQualifierOrder configuration validation and unit tests CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69764/new/ https://reviews.llvm.org/D69764 Files: clang/docs/ClangFormatStyleOptions.rst

[PATCH] D106152: [analyzer] Move test case to existing test file and remove duplicated test file.

This revision was automatically updated to reflect the committed changes. Closed by commit rG497b1b95e606: [analyzer] Move test case to existing test file and remove duplicated test file. (authored by ASDenysPetrov). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://revie

[PATCH] D104854: Introduce intrinsic llvm.isnan

sepavloff added inline comments. Comment at: llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp:6964 +return DAG.getSetCC(DL, ResultVT, Op, DAG.getConstantFP(0.0, DL, OperandVT), +ISD::SETUO); + nemanjai wrote: > sepavloff wrote: > > ne

[PATCH] D107696: [CodeComplete] Basic code completion for attribute names.

aaron.ballman added inline comments. Comment at: clang/lib/Sema/SemaCodeComplete.cpp:4390 + auto AddCompletions = [&](const ParsedAttrInfo &A) { +if (A.IsTargetSpecific && !A.existsInTarget(Context.getTargetInfo())) + return; Should we also early return

[clang] 497b57a - revert test commit

Author: Denys Petrov Date: 2021-08-10T19:19:27+03:00 New Revision: 497b57ad0b9ee34aeb8c56e9f47332912b50d58c URL: https://github.com/llvm/llvm-project/commit/497b57ad0b9ee34aeb8c56e9f47332912b50d58c DIFF: https://github.com/llvm/llvm-project/commit/497b57ad0b9ee34aeb8c56e9f47332912b50d58c.diff

[PATCH] D107841: CodeGen: No need to check for isExternC if HasStrictReturn is already false

aschwaighofer created this revision. aschwaighofer added a reviewer: rjmccall. aschwaighofer requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. NFC intended. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D107841 Files: c

[PATCH] D107837: abseil-string-find-str-contains should not propose an edit for the three-parameter version of find().

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG1fdb3e36ff37: abseil-string-find-str-contains should not propose an edit for the three… (authored by tdl-g, committed by ymandel). Repository: rG

[PATCH] D107647: [PowerPC] MMA - Remove deprecated built-ins and add new built-ins

lei added a comment. LGTM Can we update the title and description to be more specific though? [PowerPC] MMA - Rename deprecated builtins mma_assemble_acc, vsx_assemble_pair Rename deprecated builtins : __builtin_mma_assemble_acc __builtin_vsx_assemble_pair To: __builtin_mma_b

[clang-tools-extra] 1fdb3e3 - abseil-string-find-str-contains should not propose an edit for the three-parameter version of find().

Author: Tom Lokovic Date: 2021-08-10T16:39:17Z New Revision: 1fdb3e36ff379e5b3b05a00d49b6081435df727a URL: https://github.com/llvm/llvm-project/commit/1fdb3e36ff379e5b3b05a00d49b6081435df727a DIFF: https://github.com/llvm/llvm-project/commit/1fdb3e36ff379e5b3b05a00d49b6081435df727a.diff LOG: a

[PATCH] D69764: [clang-format] Add Left/Right Const fixer capability

MyDeveloperDay updated this revision to Diff 365519. MyDeveloperDay added a comment. - Rename the ConstFixer to QualifierAligmentFixer (as now we handle more than just const) and in preparation for handling others CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69764/new/ https://reviews

[PATCH] D107717: [LLVM][CMake][NFC] Resolve FIXME: Rename LLVM_CMAKE_PATH to LLVM_CMAKE_DIR throughout the project

gAlfonso-bit updated this revision to Diff 365507. gAlfonso-bit added a comment. Rebase CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107717/new/ https://reviews.llvm.org/D107717 Files: clang/CMakeLists.txt clang/lib/Basic/CMakeLists.txt compiler-rt/cmake/Modules/CompilerRTMockLL

[PATCH] D106891: [AMDGPU] [Remarks] Emit optimization remarks for FP atomics

gandhi21299 updated this revision to Diff 365522. gandhi21299 added a comment. - restricting remarks emission on AMDGPU targets only Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106891/new/ https://reviews.llvm.org/D106891 Files: clang/test/Cod

[PATCH] D106891: [AMDGPU] [Remarks] Emit optimization remarks for FP atomics

gandhi21299 added a comment. @rampitec comments? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106891/new/ https://reviews.llvm.org/D106891 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https:/

[clang] 3b39fa3 - [AIX] Define __HOS_AIX__ macro only for AIX target

Author: Jake Egan Date: 2021-08-10T13:03:17-04:00 New Revision: 3b39fa3e2815538187ed8928549a9f27c6a71ef6 URL: https://github.com/llvm/llvm-project/commit/3b39fa3e2815538187ed8928549a9f27c6a71ef6 DIFF: https://github.com/llvm/llvm-project/commit/3b39fa3e2815538187ed8928549a9f27c6a71ef6.diff LOG

[PATCH] D107825: [AIX] Define __HOS_AIX__ macro only for AIX target

This revision was automatically updated to reflect the committed changes. Closed by commit rG3b39fa3e2815: [AIX] Define __HOS_AIX__ macro only for AIX target (authored by Jake-Egan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107825/new/ https://

[PATCH] D107843: [X86] Add parentheses around casts in some of the X86 intrinsic headers.

craig.topper created this revision. craig.topper added reviewers: RKSimon, spatel, pengfei. craig.topper requested review of this revision. Herald added a project: clang. This covers the SSE and AVX/AVX2 headers. AVX512 has a lot more macros due to rounding mode. Fixes part of PR51324. Reposito

[PATCH] D107024: [DIBuilder] Do not replace empty enum types

ellis added a comment. @aprantl @dblaikie Can I get a review soon to avoid merge conflicts on these tests? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107024/new/ https://reviews.llvm.org/D107024 ___

[PATCH] D106891: [AMDGPU] [Remarks] Emit optimization remarks for FP atomics

rampitec added inline comments. Comment at: llvm/lib/CodeGen/AtomicExpandPass.cpp:628 + AI, Kind, + Remark << "A hardware CAS loop generated: if the memory is " +"known to be coarse-grain allocated then a hardware " Still the

[clang] e2ff880 - [clang][cache] Update Fuchsia-stage2.cmake to create hwasan multilibs

Author: Leonard Chan Date: 2021-08-10T10:52:24-07:00 New Revision: e2ff880a7ed464f0649c47132603562312fff102 URL: https://github.com/llvm/llvm-project/commit/e2ff880a7ed464f0649c47132603562312fff102 DIFF: https://github.com/llvm/llvm-project/commit/e2ff880a7ed464f0649c47132603562312fff102.diff

[libunwind] 08a5ac3 - libunwind: add missing break statements in EHABI

Author: Saleem Abdulrasool Date: 2021-08-10T17:53:31Z New Revision: 08a5ac36b956edeb989b4a65269a829eac26a5a2 URL: https://github.com/llvm/llvm-project/commit/08a5ac36b956edeb989b4a65269a829eac26a5a2 DIFF: https://github.com/llvm/llvm-project/commit/08a5ac36b956edeb989b4a65269a829eac26a5a2.diff

[PATCH] D107720: [analyzer] Cleanup a FIXME in SValBuilder.cpp

vabridgers updated this revision to Diff 365546. vabridgers added a comment. add test case Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107720/new/ https://reviews.llvm.org/D107720 Files: clang/lib/StaticAnalyzer/Core/SValBuilder.cpp clang/te

[PATCH] D99517: Implemented [[clang::musttail]] attribute for guaranteed tail calls.

rjmccall added a comment. It's not generically true that "anything can be tail-called if it's `noreturn`". For one, `noreturn` doesn't imply that the function doesn't exit by e.g. throwing or calling `longjmp`. For another, the most important user expectation of tail calls is that a long seri

[PATCH] D107841: CodeGen: No need to check for isExternC if HasStrictReturn is already false

rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107841/new/ https://reviews.llvm.org/D107841 ___

[PATCH] D107647: [PowerPC] MMA - Remove deprecated built-ins and add new built-ins

lei requested changes to this revision. lei added a comment. This revision now requires changes to proceed. Actually we should not be removing the deprecated bultins. Just need to add the new ones. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107

[PATCH] D107647: [PowerPC] MMA - Remove deprecated built-ins and add new built-ins

nemanjai added a comment. In D107647#2937706 , @lei wrote: > Actually we should not be removing the deprecated bultins. Just need to add > the new ones. Yes, and also the semantics are different. Repository: rG LLVM Github Monorepo CHANGES SINCE L

[PATCH] D107703: [AST][clangd] Expose documentation of Attrs on hover.

aaron.ballman added inline comments. Comment at: clang/unittests/AST/CMakeLists.txt:18 ASTVectorTest.cpp + AttrTest.cpp CommentLexer.cpp I don't think this new file got attached to the patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTI

[PATCH] D107791: [InlineAdvisor] Add single quotes around caller/callee names

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG76093b17394a: [InlineAdvisor] Add single quotes around caller/callee names (authored by MaskRay). Herald added a subscriber: emaste. Changed prior t

[PATCH] D107720: [analyzer] Cleanup a FIXME in SValBuilder.cpp

vabridgers updated this revision to Diff 365582. vabridgers added a comment. only need the one test case Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107720/new/ https://reviews.llvm.org/D107720 Files: clang/lib/StaticAnalyzer/Core/SValBuilder.

[PATCH] D107843: [X86] Add parentheses around casts in some of the X86 intrinsic headers.

spatel added a comment. Hard to see through all of the lint noise, but seems like a mechanical fix. Can we add a test like the one in the bug report? https://godbolt.org/z/sPT8e9vx9 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107843/new/ https://

[PATCH] D107703: [AST][clangd] Expose documentation of Attrs on hover.

sammccall updated this revision to Diff 365585. sammccall added a comment. Oops, forgot test file Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107703/new/ https://reviews.llvm.org/D107703 Files: clang-tools-extra/clangd/Hover.cpp clang-tools-

[PATCH] D107720: [analyzer] Cleanup a FIXME in SValBuilder.cpp

NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Awesome, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107720/new/ https://reviews.llvm.org/D107720 _

[PATCH] D107690: [Modules] Do not remove failed modules after the control block phase

vsapsai added a comment. Made the first review pass and `return Failure` makes sense to me as recovery isn't the best idea at this point. Still want to check more thoroughly if the removed code for `SUBMODULE_UMBRELLA_HEADER` and `SUBMODULE_UMBRELLA_DIR` has any load-bearing side-effects. Have

[PATCH] D107696: [CodeComplete] Basic code completion for attribute names.

sammccall added inline comments. Comment at: clang/lib/Sema/SemaCodeComplete.cpp:4390 + auto AddCompletions = [&](const ParsedAttrInfo &A) { +if (A.IsTargetSpecific && !A.existsInTarget(Context.getTargetInfo())) + return; aaron.ballman wrote: > Should w

[PATCH] D107646: [PowerPC] Fix the frame addresss computing return address for `__builtin_return_address`

NeHuang updated this revision to Diff 365470. NeHuang added a comment. Address review comments on the test case. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107646/new/ https://reviews.llvm.org/D107646 Files: llvm/lib/Target/PowerPC/PPCISelLow

[PATCH] D107138: [PowerPC] Implement cmplxl builtins

Conanap updated this revision to Diff 365596. Conanap added a comment. Removed unintended change Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107138/new/ https://reviews.llvm.org/D107138 Files: clang/lib/Basic/Targets/PPC.cpp clang/test/CodeG

[PATCH] D107703: [AST][clangd] Expose documentation of Attrs on hover.

sammccall updated this revision to Diff 365601. sammccall added a comment. Oops, I'm just bad at git. Here's the test file! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107703/new/ https://reviews.llvm.org/D107703 Files: clang-tools-extra/clang

[clang-tools-extra] 0cecb42 - [Sema] Include full range of the switch condition in -Wswitch diagnostic

Author: Sam McCall Date: 2021-08-10T22:50:40+02:00 New Revision: 0cecb42e4e13e1ce55a3568958ad104bbe6b6d6e URL: https://github.com/llvm/llvm-project/commit/0cecb42e4e13e1ce55a3568958ad104bbe6b6d6e DIFF: https://github.com/llvm/llvm-project/commit/0cecb42e4e13e1ce55a3568958ad104bbe6b6d6e.diff LO

[clang] d39ebda - [analyzer] Cleanup a FIXME in SValBuilder.cpp

Author: Vince Bridgers Date: 2021-08-10T16:12:52-05:00 New Revision: d39ebdae674c8efc84ebe8dc32716ec353220530 URL: https://github.com/llvm/llvm-project/commit/d39ebdae674c8efc84ebe8dc32716ec353220530 DIFF: https://github.com/llvm/llvm-project/commit/d39ebdae674c8efc84ebe8dc32716ec353220530.diff

[PATCH] D107720: [analyzer] Cleanup a FIXME in SValBuilder.cpp

This revision was automatically updated to reflect the committed changes. Closed by commit rGd39ebdae674c: [analyzer] Cleanup a FIXME in SValBuilder.cpp (authored by vabridgers, committed by einvbri ). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTIO

[PATCH] D107850: [WIP][asan] Implemented custom calling convention similar used by HWASan for X86. The feature should be code complete. The tests are coming in a later revision.

kstoimenov updated this revision to Diff 365608. kstoimenov added a comment. Herald added a project: clang. Herald added a subscriber: cfe-commits. Added an IR test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107850/new/ https://reviews.llvm.org

[PATCH] D107138: [PowerPC] Implement cmplxl builtins

nemanjai accepted this revision. nemanjai added a comment. This revision is now accepted and ready to land. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107138/new/ https://reviews.llvm.org/D107138 __

  1   2   >