[clang] c495dfe - [clang][cli] NFC: Decrease the scope of ParseLangArgs parameters

Author: Jan Svoboda Date: 2021-01-15T08:41:50+01:00 New Revision: c495dfe0268bc2be8737725d657411baa1399e9d URL: https://github.com/llvm/llvm-project/commit/c495dfe0268bc2be8737725d657411baa1399e9d DIFF: https://github.com/llvm/llvm-project/commit/c495dfe0268bc2be8737725d657411baa1399e9d.diff L

[PATCH] D94674: [clang][cli] NFC: Decrease the scope of ParseLangArgs parameters

This revision was automatically updated to reflect the committed changes. Closed by commit rGc495dfe0268b: [clang][cli] NFC: Decrease the scope of ParseLangArgs parameters (authored by jansvoboda11). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D9467

[clang] 1a49944 - [clang][cli] NFC: Decrease the scope of ParseCodeGenArgs parameters

Author: Jan Svoboda Date: 2021-01-15T08:42:30+01:00 New Revision: 1a49944b59dbbfd62bd860b564919087f274a5bf URL: https://github.com/llvm/llvm-project/commit/1a49944b59dbbfd62bd860b564919087f274a5bf DIFF: https://github.com/llvm/llvm-project/commit/1a49944b59dbbfd62bd860b564919087f274a5bf.diff L

[PATCH] D94675: [clang][cli] NFC: Decrease the scope of ParseCodeGenArgs parameters

This revision was automatically updated to reflect the committed changes. Closed by commit rG1a49944b59db: [clang][cli] NFC: Decrease the scope of ParseCodeGenArgs parameters (authored by jansvoboda11). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D9

[PATCH] D94724: [clangd] Remove the recovery-ast options.

hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land. Thanks, this looks good to me. I have left an explicit setting in our internal client (so this will break the build during the integration), I'll remove them. Repository: rG LLVM Github Mo

[PATCH] D94727: [clangd] Retire some flags for uncontroversial, stable features.

hokein added a comment. the code looks good to me, but we need to be a bit careful on landing this -- as we have an internal client setting this flag. Comment at: clang-tools-extra/clangd/ClangdServer.h:147 -bool SuggestMissingIncludes = false; - our int

[clang] 168be42 - [Clang] Mutate long-double math builtins into f128 under IEEE-quad

Author: Qiu Chaofan Date: 2021-01-15T16:56:20+08:00 New Revision: 168be4208304e36d3bb156b5c413b340a391383e URL: https://github.com/llvm/llvm-project/commit/168be4208304e36d3bb156b5c413b340a391383e DIFF: https://github.com/llvm/llvm-project/commit/168be4208304e36d3bb156b5c413b340a391383e.diff L

[PATCH] D92080: [Clang] Mutate long-double math builtins into f128 under IEEE-quad

This revision was automatically updated to reflect the committed changes. Closed by commit rG168be4208304: [Clang] Mutate long-double math builtins into f128 under IEEE-quad (authored by qiucf). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92080/new

[PATCH] D94673: [analyzer][CTU] API for CTU macro expansions

steakhal updated this revision to Diff 316861. steakhal edited the summary of this revision. steakhal set the repository for this revision to rG LLVM Github Monorepo. steakhal added a comment. Herald added a reviewer: shafik. Herald added a project: clang. Herald added a subscriber: cfe-commits. U

[PATCH] D94673: [analyzer][CTU] API for CTU macro expansions

vsavchenko added inline comments. Comment at: clang/include/clang/CrossTU/CrossTranslationUnit.h:186 - /// Determine the original source location in the original TU for an - /// imported source location. + /// Returns the MacroExpansionContext for the imported TU to which th

[PATCH] D94753: [clangd] exclude symbols from document outline which do not originate from the main file

kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land. thanks for the fix, a small comment about testing though. Comment at: clang-tools-extra/clangd/unittests/FindSymbolsTests.cpp:532 TEST(DocumentSymbols, InHeaderFile) {

[PATCH] D94673: [analyzer][CTU] API for CTU macro expansions

steakhal added inline comments. Comment at: clang/include/clang/CrossTU/CrossTranslationUnit.h:191 + /// source-location, empty is returned. + /// \note Macro expansion tracking for imported TUs are not implemented yet. + /// It returns empty unconditionally. -

[PATCH] D70401: [WIP][RISCV] Implement ilp32e ABI

lenary marked 3 inline comments as done. lenary added inline comments. Comment at: llvm/lib/Target/RISCV/RISCVRegisterInfo.cpp:104 + // so that we don't insert `fp` manipulation code into functions that do not + // require it. + const MachineFrameInfo &MFI = MF.getFrameInfo();

[PATCH] D93401: [flang][driver] Add support for `-D`, `-U`

FarisRehman marked an inline comment as done. FarisRehman added inline comments. Comment at: flang/test/Flang-Driver/driver-help.f90:22 ! HELP-NEXT: -### Print (but do not run) the commands to run for this compilation +! HELP-NEXT: -D = Define to (or 1 i

[PATCH] D94753: [clangd] exclude symbols from document outline which do not originate from the main file

ilya-golovenko updated this revision to Diff 316877. ilya-golovenko added a comment. Update existing test and remove redundant one Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94753/new/ https://reviews.llvm.org/D94753 Files: clang-tools-extra/

[PATCH] D94753: [clangd] exclude symbols from document outline which do not originate from the main file

ilya-golovenko added a comment. @kadircet Thank you for review! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94753/new/ https://reviews.llvm.org/D94753 ___ cfe-commits mailing list cfe-commits@lists.ll

[PATCH] D94755: [clangd] Fix division by zero when computing scores

kadircet created this revision. kadircet added a reviewer: sammccall. Herald added subscribers: usaxena95, arphaman. kadircet requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang. NameMatch could be a float close to zero, i

[PATCH] D94753: [clangd] exclude symbols from document outline which do not originate from the main file

kadircet added a comment. btw, do you have commit access or should i land this for you ? (if so please provide your email) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94753/new/ https://reviews.llvm.org/D94753 __

[PATCH] D94621: [clang-tidy] add concurrency-async-fs

segoon added a comment. Eugene.Zelenko, thanks for the review! fixed CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94621/new/ https://reviews.llvm.org/D94621 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-b

[PATCH] D94622: [clang-tidy] add concurrency-async-no-new-threads

segoon added a comment. Eugene.Zelenko, thanks for the review! fixed CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94622/new/ https://reviews.llvm.org/D94622 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-b

[PATCH] D94753: [clangd] exclude symbols from document outline which do not originate from the main file

ilya-golovenko added a comment. In D94753#2500535 , @kadircet wrote: > btw, do you have commit access or should i land this for you ? (if so please > provide your email) I've got commit access some time ago. Thanks again! Repository: rG LLVM Github M

[clang-tools-extra] 9cc221b - [clangd] exclude symbols from document outline which do not originate from the main file

Author: Ilya Golovenko Date: 2021-01-15T13:23:12+03:00 New Revision: 9cc221b99becf20397d935981eeb48cba5be7faf URL: https://github.com/llvm/llvm-project/commit/9cc221b99becf20397d935981eeb48cba5be7faf DIFF: https://github.com/llvm/llvm-project/commit/9cc221b99becf20397d935981eeb48cba5be7faf.diff

[PATCH] D94753: [clangd] exclude symbols from document outline which do not originate from the main file

This revision was automatically updated to reflect the committed changes. Closed by commit rG9cc221b99bec: [clangd] exclude symbols from document outline which do not originate from the… (authored by ilya-golovenko). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://revie

[PATCH] D94606: [clangd] Move DirBasedCDB broadcasting onto its own thread.

kadircet added inline comments. Comment at: clang-tools-extra/clangd/GlobalCompilationDatabase.cpp:496 + std::condition_variable CV; + std::atomic ShouldStop = {false}; // Must notify CV after writing. + std::deque Queue; nit: we already hold the lock within t

[PATCH] D82862: [ThinLTO] Always parse module level inline asm with At&t dialect

uweigand added a comment. In D82862#2500038 , @pengfei wrote: >> What is the reason for treating this differently in LLVM? > > I'm not sure if it is related to this. I think one difference is that LLVM is > supporting MS inline assembly. Although it uses

[PATCH] D75594: [AArch64] Add support for Fujitsu A64FX

kawashima-fj closed this revision. kawashima-fj added a comment. Herald added subscribers: dexonsmith, danielkiss. In D75594#1927988 , @ikitayama wrote: > In D75594#1927959 , @kawashima-fj > wrote: > >> Yes, https:

[PATCH] D82862: [ThinLTO] Always parse module level inline asm with At&t dialect

hans added a subscriber: rnk. hans added a comment. > In D82862#2498785 , @hans wrote: > >> The motivation for my change was really just to make ThinLTO compiles work >> the same as non-ThinLTO ones. >> >> Maybe the fact that -x86-asm-syntax=intel doesn't

[PATCH] D94697: [clangd] Update CC Ranking model with better sampling.

adamcz accepted this revision. adamcz added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/Quality.cpp:515 + E.setFractionNameInContext( + Relevance.ContextWords ? NumMatch * 1.0 / Relevance.ContextWords->size() +

[PATCH] D94127: [ASTMatchers] Add mapAnyOf matcher

aaron.ballman accepted this revision. aaron.ballman added a comment. LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94127/new/ https://reviews.llvm.org/D94127 ___ cfe-commits mailing list cfe-commit

[PATCH] D94128: [ASTMatchers] Make cxxOperatorCallExpr matchers API-compatible with n-ary operators

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM Comment at: clang/include/clang/ASTMatchers/ASTMatchersInternal.h:1994 + return None; +return FD->getNumParams() > 0 ? UO_PostInc : UO_PreInc; +

[PATCH] D94129: [ASTMatchers] Add binaryOperation matcher

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM aside from some minor NFC improvements. Comment at: clang/lib/ASTMatchers/Dynamic/Marshallers.h:957 + + void getArgKinds(ASTNodeKind ThisKind, unsigned Ar

[PATCH] D93453: [flang][driver] Add support for `-I`

FarisRehman marked 2 inline comments as done. FarisRehman added inline comments. Comment at: flang/test/Flang-Driver/include-header.f90:38 +!- +! EXPECTED OUTPUT FOR /Inputs/ FOLDER SPECIFIED FIRST +!

[PATCH] D93453: [flang][driver] Add support for `-I`

FarisRehman updated this revision to Diff 316908. FarisRehman marked 2 inline comments as done. FarisRehman added a comment. Add include-module test Update the regression test to check the behaviour of -I with an INCLUDE line in the source code. Also add a regression test to check the behaviour

[PATCH] D94130: [ASTMatchers] Add support for CXXRewrittenBinaryOperator

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM aside from the `auto` usage. Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:5346 +AST_POLYMORPHIC_SUPPORTED_TYPES(BinaryOperator, CXXOperato

[PATCH] D94473: [clangd] Use AST-based signals in CodeCompletion.

adamcz added inline comments. Comment at: clang-tools-extra/clangd/Quality.cpp:349 + }; + DeriveASTSignals(); // Declarations are scoped, others (like macros) are assumed global. Maybe it's just me, but using a lambda like this seems quite confusing. I don't

[clang-tools-extra] aeaeb9e - [clangd] Make ExpandAutoType not available on template params.

Author: Adam Czachorowski Date: 2021-01-15T14:19:05+01:00 New Revision: aeaeb9e6bdc90d9c4b839ac0e4edc6255021cced URL: https://github.com/llvm/llvm-project/commit/aeaeb9e6bdc90d9c4b839ac0e4edc6255021cced DIFF: https://github.com/llvm/llvm-project/commit/aeaeb9e6bdc90d9c4b839ac0e4edc6255021cced.d

[PATCH] D94719: [clangd] Make ExpandAutoType not available on template params.

This revision was automatically updated to reflect the committed changes. Closed by commit rGaeaeb9e6bdc9: [clangd] Make ExpandAutoType not available on template params. (authored by adamcz). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94719/new/

[PATCH] D94624: PATCH] [clang-query] Add a --use-color option to clang-query to allow forcing the behavior

aaron.ballman added a comment. Thanks for this, I think it's looking more promising! I'd still like to see some test coverage for the changes so long as it doesn't turn into a slog. Tests like `clang\test\AST\ast-dump-color.cpp` show roughly how it's done in the frontend. If it turns out that t

[clang-tools-extra] c77c3d1 - [clangd] Set correct CWD when using compile_flags.txt

Author: Adam Czachorowski Date: 2021-01-15T14:26:24+01:00 New Revision: c77c3d1d18cdd58989f9d35bbf6c31f5fda0a125 URL: https://github.com/llvm/llvm-project/commit/c77c3d1d18cdd58989f9d35bbf6c31f5fda0a125 DIFF: https://github.com/llvm/llvm-project/commit/c77c3d1d18cdd58989f9d35bbf6c31f5fda0a125.d

[PATCH] D94699: [clangd] Set correct CWD when using compile_flags.txt

This revision was automatically updated to reflect the committed changes. Closed by commit rGc77c3d1d18cd: [clangd] Set correct CWD when using compile_flags.txt (authored by adamcz). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94699/new/ https://r

[PATCH] D90448: [clang] Add type check for explicit instantiation of static data members

lebedev.ri added a comment. Tests missing Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90448/new/ https://reviews.llvm.org/D90448 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.ll

[PATCH] D90448: [clang] Add type check for explicit instantiation of static data members

nomanous added a comment. In D90448#2500829 , @lebedev.ri wrote: > Tests missing I mistakenly missed that file in the last commit. I'll add it soon. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90448/new/

[PATCH] D92277: [OpenCL] Refactor of targets OpenCL option settings

Anastasia added a comment. This patch contains very useful improvements to the design along with the behavioral fixes. I see that it opens more opportunities to bring the implementation in line with the conventional flow and therefore simplify the further development of new functionality. More

[clang] d1862a1 - [OpenCL][Docs] Fixed malformed table in OpenCLSupport

Author: Anastasia Stulova Date: 2021-01-15T14:27:26Z New Revision: d1862a16310379179a40b309a9721318ae7e3254 URL: https://github.com/llvm/llvm-project/commit/d1862a16310379179a40b309a9721318ae7e3254 DIFF: https://github.com/llvm/llvm-project/commit/d1862a16310379179a40b309a9721318ae7e3254.diff

[clang] 791634b - [clang][cli] Parse & generate options necessary for LangOptions defaults manually

Author: Jan Svoboda Date: 2021-01-15T15:38:43+01:00 New Revision: 791634b999e33e029aeeda91eeb5fae13757dcdc URL: https://github.com/llvm/llvm-project/commit/791634b999e33e029aeeda91eeb5fae13757dcdc DIFF: https://github.com/llvm/llvm-project/commit/791634b999e33e029aeeda91eeb5fae13757dcdc.diff L

[PATCH] D94678: [clang][cli] Parse & generate options necessary for LangOptions defaults manually

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG791634b999e3: [clang][cli] Parse & generate options necessary for LangOptions defaults… (authored by jansvoboda11). Changed prior to commit: https

[PATCH] D94779: [Clang] Ensure vector predication pragma is ignored only when vectorization width is 1.

malharJ created this revision. Herald added a subscriber: rogfer01. malharJ requested review of this revision. Herald added subscribers: cfe-commits, vkmr. Herald added a project: clang. This ensures that the Clang loop pragma vectorize_predicate([enable|disable]) is ignored when vectorize_width(

[PATCH] D84846: [MC] Add support for generating missing GNU build notes

tbaeder updated this revision to Diff 316944. Herald added a subscriber: martong. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84846/new/ https://reviews.llvm.org/D84846 Files: clang/include/clang/AST/Expr.h clang/include/clang/AST/RecursiveASTVisitor.h clang/include/clang/Basic/S

[PATCH] D84846: [MC] Add support for generating missing GNU build notes

tbaeder updated this revision to Diff 316945. tbaeder added a comment. Sorry about the last change, that was the wrong patch. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84846/new/ https://reviews.llvm.org/D84846 Files: clang/include/clang/Basic/CodeGenOptions.def clang/include/c

[PATCH] D94779: [Clang] Ensure vector predication pragma is ignored only when vectorization width is 1.

SjoerdMeijer added inline comments. Comment at: clang/test/CodeGenCXX/pragma-loop-predicate.cpp:102 + +// CHECK-NEXT: ![[LOOP7]] = distinct !{![[LOOP7]], [[MP]], [[GEN8]], [[GEN11:![0-9]+]], [[GEN3]]} +// CHECK-NEXT: [[GEN11]] = !{!"llvm.loop.vectorize.width", i32 4} ---

[PATCH] D94779: [Clang] Ensure vector predication pragma is ignored only when vectorization width is 1.

malharJ added inline comments. Comment at: clang/test/CodeGenCXX/pragma-loop-predicate.cpp:102 + +// CHECK-NEXT: ![[LOOP7]] = distinct !{![[LOOP7]], [[MP]], [[GEN8]], [[GEN11:![0-9]+]], [[GEN3]]} +// CHECK-NEXT: [[GEN11]] = !{!"llvm.loop.vectorize.width", i32 4}

[clang] 3832629 - [clang][cli] NFC: Add PIE parsing for precompiled input and IR

Author: Jan Svoboda Date: 2021-01-15T16:41:34+01:00 New Revision: 383262933045e1c138362105be4ee4d1b62ab4cc URL: https://github.com/llvm/llvm-project/commit/383262933045e1c138362105be4ee4d1b62ab4cc DIFF: https://github.com/llvm/llvm-project/commit/383262933045e1c138362105be4ee4d1b62ab4cc.diff L

[clang] a7dcd3a - [clang][cli] NFC: Parse some LangOpts after the defaults are set

Author: Jan Svoboda Date: 2021-01-15T16:41:34+01:00 New Revision: a7dcd3aeb0fb58ad774bc89428ed6c925f31f8aa URL: https://github.com/llvm/llvm-project/commit/a7dcd3aeb0fb58ad774bc89428ed6c925f31f8aa DIFF: https://github.com/llvm/llvm-project/commit/a7dcd3aeb0fb58ad774bc89428ed6c925f31f8aa.diff L

[clang] 1744f4c - [clang][cli] NFC: Promote ParseLangArgs and ParseCodeGenArgs to members

Author: Jan Svoboda Date: 2021-01-15T16:41:34+01:00 New Revision: 1744f4c676411ebd2e38afd5a6b56e5dd533c6ac URL: https://github.com/llvm/llvm-project/commit/1744f4c676411ebd2e38afd5a6b56e5dd533c6ac DIFF: https://github.com/llvm/llvm-project/commit/1744f4c676411ebd2e38afd5a6b56e5dd533c6ac.diff L

[PATCH] D94679: [clang][cli] NFC: Add PIE parsing for precompiled input and IR

This revision was automatically updated to reflect the committed changes. Closed by commit rG383262933045: [clang][cli] NFC: Add PIE parsing for precompiled input and IR (authored by jansvoboda11). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94679/

[PATCH] D94680: [clang][cli] NFC: Parse some LangOpts after the defaults are set

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGa7dcd3aeb0fb: [clang][cli] NFC: Parse some LangOpts after the defaults are set (authored by jansvoboda11). Repository: rG LLVM Github Monorepo CH

[PATCH] D94681: [clang][cli] NFC: Promote ParseLangArgs and ParseCodeGenArgs to members

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG1744f4c67641: [clang][cli] NFC: Promote ParseLangArgs and ParseCodeGenArgs to members (authored by jansvoboda11). Repository: rG LLVM Github Monor

[PATCH] D94614: [FPEnv][X86] Platform builtins edition: clang should get from the AST the metadata for constrained FP builtins

pengfei added a comment. > This doesn't add metadata to llvm intrinsics that are not constrained. Oh, right. I misunderstood what's doing in these patches and thought we can add metadata to any intrinsics by CGFPOptionsRAII now. :-) > If a relevant #pragma is used then without this change the m

[PATCH] D94785: [clangd] Index local classes, virtual and overriding methods.

usaxena95 created this revision. Herald added subscribers: kadircet, arphaman. usaxena95 requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang. Previously we did not record local class declarations. Now with features like fi

[PATCH] D94697: [clangd] Update CC Ranking model with better sampling.

usaxena95 updated this revision to Diff 316969. usaxena95 added a comment. Addressed comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94697/new/ https://reviews.llvm.org/D94697 Files: clang-tools-extra/clangd/Quality.cpp clang-tools-ext

[PATCH] D94786: [clang][ASTImporter] Add support for importing CXXFoldExpr.

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

[PATCH] D94785: [clangd] Index local classes, virtual and overriding methods.

njames93 added a comment. Thank you for taking the time to fix this and the performance measurements to boot. My only question is does this try and index lambdas, under the hood they are declared as a CXXRecordDecl, defined in function scope? Repository: rG LLVM Github Monorepo CHANGES SINC

[PATCH] D94787: [clang][AST] Add get functions for CXXFoldExpr paren locations.

balazske created this revision. Herald added subscribers: martong, gamesh411, Szelethus, dkrupp. balazske 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/D94787 Files: clang

[PATCH] D94786: [clang][ASTImporter] Add support for importing CXXFoldExpr.

balazske added a reviewer: martong. balazske added a comment. Herald added a subscriber: rnkovacs. There is a related bug report: https://bugs.llvm.org/show_bug.cgi?id=48004 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94786/new/ https://reviews.l

[clang-tools-extra] d5047d7 - [clangd] Update CC Ranking model with better sampling.

Author: Utkarsh Saxena Date: 2021-01-15T18:13:24+01:00 New Revision: d5047d762f391c94939d67fc84cae25b24125694 URL: https://github.com/llvm/llvm-project/commit/d5047d762f391c94939d67fc84cae25b24125694 DIFF: https://github.com/llvm/llvm-project/commit/d5047d762f391c94939d67fc84cae25b24125694.diff

[PATCH] D94697: [clangd] Update CC Ranking model with better sampling.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGd5047d762f39: [clangd] Update CC Ranking model with better sampling. (authored by usaxena95). Repository: rG LLVM Github Monorepo CHANGES SINCE L

[clang] bc84f89 - [OpenCL][Docs] Fixed cross-section reference in OpenCLSupport

Author: Anastasia Stulova Date: 2021-01-15T17:20:13Z New Revision: bc84f89c71ab62d510973f64f022bee31e53af96 URL: https://github.com/llvm/llvm-project/commit/bc84f89c71ab62d510973f64f022bee31e53af96 DIFF: https://github.com/llvm/llvm-project/commit/bc84f89c71ab62d510973f64f022bee31e53af96.diff

[PATCH] D94802: [clang][cli] Parse HeaderSearch options separately

jansvoboda11 created this revision. jansvoboda11 added reviewers: dexonsmith, Bigcheese. Herald added a subscriber: dang. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.ll

[clang] 6227069 - [DebugInfo][CodeView] Change in line tables only mode to emit type information

Author: Amy Huang Date: 2021-01-15T09:28:27-08:00 New Revision: 6227069bdce6b0c3c22f0a0c8f1aef705985125a URL: https://github.com/llvm/llvm-project/commit/6227069bdce6b0c3c22f0a0c8f1aef705985125a DIFF: https://github.com/llvm/llvm-project/commit/6227069bdce6b0c3c22f0a0c8f1aef705985125a.diff LOG

[PATCH] D94639: [DebugInfo][CodeView] Change in line tables only mode to emit parent/context scopes for functions, using declarations for types

This revision was automatically updated to reflect the committed changes. Closed by commit rG6227069bdce6: [DebugInfo][CodeView] Change in line tables only mode to emit type information (authored by akhuang). Changed prior to commit: https://reviews.llvm.org/D94639?vs=316779&id=317001#toc Repo

[PATCH] D94803: [clang][cli] Generate HeaderSearch options separately

jansvoboda11 created this revision. jansvoboda11 added reviewers: dexonsmith, Bigcheese. jansvoboda11 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/D94803 Files: clang/lib

[PATCH] D94472: [clang][cli] Manually generate header search arguments

jansvoboda11 updated this revision to Diff 317003. jansvoboda11 added a comment. Herald added subscribers: llvm-commits, hiraditya. Herald added a project: LLVM. Add (improved) round-trip mechanism Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94472

[PATCH] D94804: [clang] Allow LifetimeExtendedTemporary to have no access specifier

adamcz created this revision. adamcz added a reviewer: hokein. adamcz requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. The check only runs in debug mode during serialization, but assert()-fail on: struct S { const int& x = 7; }; in C++ m

[PATCH] D94472: [WIP][clang][cli] Command line round-trip for HeaderSearch options

jansvoboda11 added inline comments. Comment at: clang/lib/Frontend/CompilerInvocation.cpp:1839 +OptName = Dash + OptTable.getOptionName(OPT_stdlib_EQ); +Args.emplace_back(SA(OptName + "libc++")); + } I plan to move the prefixing of the option name and se

[PATCH] D93264: [CSSPGO] Introducing distribution factor for pseudo probe.

wmi accepted this revision. wmi added a comment. This revision is now accepted and ready to land. LGTM. Thanks. Comment at: llvm/lib/Transforms/IPO/SampleProfile.cpp:363 const FunctionSamples *CalleeSamples; uint64_t CallsiteCount; + // Call site distribution factor to p

[PATCH] D94474: [WIP][clang][cli] Test manual header search argument generation with round-trip

jansvoboda11 abandoned this revision. jansvoboda11 added a comment. Hmm, that would still require us to keep a list of options we've converted. It's better than hard-coding it like this patch does, but I'd like it to be more automatic. I've updated D94472 with

[PATCH] D94806: [OpenMP] Add support for mapping names in mapper API

jhuber6 created this revision. jhuber6 added a reviewer: jdoerfert. Herald added subscribers: guansong, yaxunl. jhuber6 requested review of this revision. Herald added subscribers: llvm-commits, openmp-commits, cfe-commits, sstefan1. Herald added projects: clang, OpenMP, LLVM. The custom mapper AP

[PATCH] D94808: [NewPM][Inliner] Move mandatory inliner inside function simplification pipeline

aeubanks created this revision. Herald added subscribers: hiraditya, eraman. aeubanks requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D94808 Files: clang/t

[PATCH] D94808: [NewPM][Inliner] Move mandatory inliner inside function simplification pipeline

aeubanks added a comment. Wondering if this makes sense to you (before I go and finish cleaning up all the tests) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94808/new/ https://reviews.llvm.org/D94808 ___

[PATCH] D94806: [OpenMP] Add support for mapping names in mapper API

jdoerfert accepted this revision. jdoerfert 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/D94806/new/ https://reviews.llvm.org/D94806 ___

[PATCH] D94785: [clangd] Index local classes, virtual and overriding methods.

usaxena95 updated this revision to Diff 317015. usaxena95 added a comment. Added tests for not indexing lambdas. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94785/new/ https://reviews.llvm.org/D94785 Files: clang-tools-extra/clangd/index/FileI

[PATCH] D94785: [clangd] Index local classes, virtual and overriding methods.

usaxena95 added a comment. In D94785#2501312 , @njames93 wrote: > My only question is does this try and index lambdas, under the hood they are > declared as a CXXRecordDecl, defined in function scope? I would not expect that to happen as it is an anonymo

[PATCH] D94808: [NewPM][Inliner] Move mandatory inliner inside function simplification pipeline

mtrofin added a comment. I think InlineAdvisor::getAdvice needs to take a bool MandatoryOnly, which Inliner then passes. This allows us to then use the same advisor for both the always case and the policy-driven inliner instances, which allows that advisor to correctly book-keep any module wide

[PATCH] D94808: [NewPM][Inliner] Move mandatory inliner inside function simplification pipeline

aeubanks added inline comments. Comment at: llvm/lib/Transforms/IPO/Inliner.cpp:661 FunctionAnalysisManager &FAM, Module &M) { + if (Mandatory) { +OwnedAdvisor = std::make_unique(FAM); mtrofin wrote: > This should move to line 675: i

[PATCH] D94624: PATCH] [clang-query] Add a --use-color option to clang-query to allow forcing the behavior

tomrittervg updated this revision to Diff 317027. tomrittervg added a comment. Remove braces Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94624/new/ https://reviews.llvm.org/D94624 Files: clang-tools-extra/clang-query/Query.cpp clang-tools-ex

[PATCH] D94624: PATCH] [clang-query] Add a --use-color option to clang-query to allow forcing the behavior

tomrittervg added a comment. I started trying to work on a patch, but I'm still unpacking from a move and don't have all my machines set up - trying to enable and build tests filled up my hard drive (even after I removed the easy-to-remove stuff), so I don't think I'm going to be able to create

[PATCH] D94806: [OpenMP] Add support for mapping names in mapper API

jhuber6 updated this revision to Diff 317038. jhuber6 added a comment. Fixed test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94806/new/ https://reviews.llvm.org/D94806 Files: clang/lib/CodeGen/CGOpenMPRuntime.cpp clang/test/OpenMP/declare_m

[PATCH] D94814: [HIP] Support `__managed__` attribute

yaxunl created this revision. yaxunl added a reviewer: tra. Herald added subscribers: dexonsmith, jdoerfert, hiraditya, mgorny. Herald added a reviewer: aaron.ballman. yaxunl requested review of this revision. Herald added a project: LLVM. Herald added a subscriber: llvm-commits. This patch implem

[clang] a1be47b - [CodeView][DebugInfo] Add test case to show that linkage names are not

Author: Amy Huang Date: 2021-01-15T12:05:33-08:00 New Revision: a1be47b4771467998d7549dcd1b9f9cebdaa9af9 URL: https://github.com/llvm/llvm-project/commit/a1be47b4771467998d7549dcd1b9f9cebdaa9af9 DIFF: https://github.com/llvm/llvm-project/commit/a1be47b4771467998d7549dcd1b9f9cebdaa9af9.diff LOG

[PATCH] D94820: Support for instrumenting only selected files or functions

phosek created this revision. phosek added reviewers: davidxl, vsk, gulfem. Herald added subscribers: dexonsmith, wenlei, dang, jdoerfert, steven_wu, hiraditya, mgorny. phosek requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits

[PATCH] D94472: [WIP][clang][cli] Command line round-trip for HeaderSearch options

dexonsmith added a comment. A concern I have is that the round-tripping might be too low level since it requires each group of options to opt-in somehow. Having something at the top-level that doesn't have to be touched again would be easier to validate / understand, and might better serve the

[PATCH] D85223: [CUDA][HIP] Support accessing static device variable in host code for -fgpu-rdc

yaxunl added a comment. In D85223#2452363 , @JonChesterfield wrote: > I concede that making the variables external, and trying to give them unique > names, does work around static variables not working. I believe static > variables are subjected to more

[clang] 4a47da2 - [Sema] turns -Wfree-nonheap-object on by default

Author: Christopher Di Bella Date: 2021-01-15T21:38:47Z New Revision: 4a47da2cf440c2f2006d9b04acfef4292de1e263 URL: https://github.com/llvm/llvm-project/commit/4a47da2cf440c2f2006d9b04acfef4292de1e263 DIFF: https://github.com/llvm/llvm-project/commit/4a47da2cf440c2f2006d9b04acfef4292de1e263.dif

[PATCH] D94786: [clang][ASTImporter] Add support for importing CXXFoldExpr.

shafik added inline comments. Comment at: clang/unittests/AST/ASTImporterTest.cpp:655 + Lang_CXX17, "", Lang_CXX17, Verifier, + functionTemplateDecl(hasDescendant(cxxFoldExpr(; +} Is it possible to check that we imported four fold expr

[PATCH] D94825: [NewPM]i[Inliner] Move the 'always inliner' case in the same CGSCC pass as 'regular' inliner

mtrofin created this revision. mtrofin added a reviewer: aeubanks. Herald added subscribers: hiraditya, eraman. mtrofin requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. Expanding from D94808

[PATCH] D94827: [SimplifyCFG] Require and preserve dominator tree

lebedev.ri created this revision. lebedev.ri added reviewers: fhahn, jdoerfert, arsenm, mkazantsev, kuhar, brzycki, nikic. lebedev.ri added projects: LLVM, AMDGPU. Herald added subscribers: wenlei, kerbowa, steven_wu, hiraditya, nhaehnle, jvesely. lebedev.ri requested review of this revision. Her

[PATCH] D94829: [NFC] Add -std=c11 to attr-availability.c

justice_adams created this revision. justice_adams added reviewers: nigelp-xmos, aaron.ballman. justice_adams added a project: clang. justice_adams requested review of this revision. This test will fail with any toolchains that don't default to C11. Adding this switch to the clang invocation in t

[PATCH] D94735: CGDebugInfo CreatedLimitedType: Drop file/line for RecordType with invalid location

dblaikie accepted this revision. dblaikie added a comment. This revision is now accepted and ready to land. Couple of optional pieces. Comment at: clang/lib/CodeGen/CGDebugInfo.cpp:3338-3340 + const SourceLocation Loc = RD->getLocation(); + llvm::DIFile *DefUnit = Loc.isValid

[PATCH] D94829: [NFC] Add -std=c11 to attr-availability.c

dyung accepted this revision. dyung 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/D94829/new/ https://reviews.llvm.org/D94829 ___ cfe

[PATCH] D93776: [clang-format] Add StatementAttributeLikeMacros option

HazardyKnusperkeks added a comment. *ping* And I don't have a better name. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93776/new/ https://reviews.llvm.org/D93776 ___ cfe-commits mailing list cfe-commit

  1   2   >