[PATCH] D73282: Fix debug-info generation for block invocations so that we set the LinkageName instead of the Name

shafik marked an inline comment as done. shafik added inline comments. Comment at: clang/lib/CodeGen/CGDebugInfo.cpp:3659 // Use llvm function name. -Name = Fn->getName(); +if (Fn->getName().startswith("___Z")) + LinkageName = Fn->getName();

[PATCH] D73462: [dwarf-5] Support DebugInfo for Defaulted parameters for C++ templates

awpandey marked an inline comment as not done. awpandey added a comment. @aprantl Thanks for your valuable feedback. I have a doubt. You need a .ll file its corresponding llvm bytecode generated by current llvm (without my patch). Doest this ll file includes any checks? In the round trip test

[PATCH] D73282: Fix debug-info generation for block invocations so that we set the LinkageName instead of the Name

shafik updated this revision to Diff 241638. shafik added a comment. Using a more targeted approach. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73282/new/ https://reviews.llvm.org/D73282 Files: clang/lib/CodeGen/CGDebugInfo.cpp

[PATCH] D72841: [RFC] Add support for pragma float_control, to control precision and exception behavior at the source level

sepavloff added a comment. You could check the scope where the pragma appears is the same way as `pragma clang fp` does. The code case tok::annot_pragma_fp: ProhibitAttributes(Attrs); Diag(Tok, diag::err_pragma_fp_scope); ConsumeAnnotationToken(); return StmtError(); is put

[PATCH] D73762: [clang] New warning for for-loops where the iteration does not match the loop condition

Eugene.Zelenko added a comment. It's good question where this check belongs. There are two loop related checks in CLang-tidy: bugprone-infinite-loop and bugprone-too-small-loop-variable. Comment at: clang/lib/Sema/SemaStmt.cpp:1754 +if (!Cond->isRelationalOp()) return; +

[PATCH] D71227: [cuda][hip] Fix function overload resolution in the global initiailizer.

hliao added a comment. In D71227#1831445 , @hliao wrote: > Sorry for the late reply. Really appreciate your feedback. Thanks! @rsmith Have you chance to review the revised change? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] 1db66e7 - PR44627: Consider reversing == and <=> candidates found by ADL.

Author: Richard Smith Date: 2020-01-30T18:41:54-08:00 New Revision: 1db66e705f4dbe7dbe17edac804289ef59d5f616 URL: https://github.com/llvm/llvm-project/commit/1db66e705f4dbe7dbe17edac804289ef59d5f616 DIFF: https://github.com/llvm/llvm-project/commit/1db66e705f4dbe7dbe17edac804289ef59d5f616.diff

[PATCH] D73762: New warning for for-loops where the iteration does not match the loop condition

rtrieu created this revision. This is a warning for when the increment/decrement in a for loop does not match the condition in the loop. If the condition has a relational operator, one operand can be deduced to be the larger value and the other operand the smaller value when the loop is run.

[PATCH] D73701: [clang] fix linkage of nested lambda

rsmith added a comment. This is effectively reverting https://reviews.llvm.org/D1783. If we don't need that any more for whatever reason, can we also get rid of the check for whether the outermost lambda has a mangling number a few lines above? (Please also check if we can remove

[PATCH] D73208: [objc_direct] fix codegen for mismatched Decl/Impl return types

This revision was automatically updated to reflect the committed changes. Closed by commit rG6eb969b7c5b5: [objc_direct] fix codegen for mismatched Decl/Impl return types (authored by MadCoder). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] c6cf360 - Revert "[objc_direct] Small updates to help with adoption."

Author: Pierre Habouzit Date: 2020-01-30T18:21:25-08:00 New Revision: c6cf3602e27b76669f785f5fd34b928a889b9db5 URL: https://github.com/llvm/llvm-project/commit/c6cf3602e27b76669f785f5fd34b928a889b9db5 DIFF:

[clang] 6eb969b - [objc_direct] fix codegen for mismatched Decl/Impl return types

Author: Pierre Habouzit Date: 2020-01-30T18:17:45-08:00 New Revision: 6eb969b7c5b5195d7f85b70e2e060eb2989cdc3c URL: https://github.com/llvm/llvm-project/commit/6eb969b7c5b5195d7f85b70e2e060eb2989cdc3c DIFF:

[clang] bebb8e2 - [objc_direct] Small updates to help with adoption.

Author: Pierre Habouzit Date: 2020-01-30T18:17:45-08:00 New Revision: bebb8e2596af8fe97f9b6356ff06e4e80580cf32 URL: https://github.com/llvm/llvm-project/commit/bebb8e2596af8fe97f9b6356ff06e4e80580cf32 DIFF:

[clang] 980517b - [Concepts] Check function constraints before deducing auto return type

Author: Saar Raz Date: 2020-01-31T03:51:26+02:00 New Revision: 980517b3530ffb7faa1a23fdc007d78f5b45ae3c URL: https://github.com/llvm/llvm-project/commit/980517b3530ffb7faa1a23fdc007d78f5b45ae3c DIFF: https://github.com/llvm/llvm-project/commit/980517b3530ffb7faa1a23fdc007d78f5b45ae3c.diff

[PATCH] D73245: Don't define std::max_align_t if not used in C++03 mode

rsmith added a comment. libc++ intentionally provides all the C++11 library functionality that it can, even when used from C++03 mode. So on the face of it, providing this name in C++03 mode seems appropriate. However... the implementation currently used by libc++ **doesn't work** in that

[clang] 5ae6554 - PR41991: Accept attributes on defaulted and deleted friends.

Author: Richard Smith Date: 2020-01-30T17:42:17-08:00 New Revision: 5ae6554a1dcd2e39346030c06d364492901c9e8d URL: https://github.com/llvm/llvm-project/commit/5ae6554a1dcd2e39346030c06d364492901c9e8d DIFF: https://github.com/llvm/llvm-project/commit/5ae6554a1dcd2e39346030c06d364492901c9e8d.diff

[PATCH] D73755: [objc_direct] Small updates to help with adoption.

MadCoder created this revision. MadCoder added reviewers: ahatanak, dexonsmith, arphaman, erik.pilkington. MadCoder added a project: clang. Herald added subscribers: cfe-commits, jfb. Add fixits for messaging self in MRR or using super, as the intent is clear, and it turns out people do that a

[PATCH] D73208: [objc_direct] fix codegen for mismatched Decl/Impl return types

MadCoder updated this revision to Diff 241613. MadCoder marked an inline comment as done. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73208/new/ https://reviews.llvm.org/D73208 Files: clang/lib/CodeGen/CGObjCMac.cpp clang/test/CodeGenObjC/direct-method-ret-mismatch.m Index:

Re: [clang] 1f3f8c3 - PR44721: Don't consider overloaded operators for built-in comparisons

Hi Hans, This is a pretty safe bugfix for a new feature in Clang 10. OK for branch? On Thu, 30 Jan 2020 at 17:17, Richard Smith via cfe-commits < cfe-commits@lists.llvm.org> wrote: > > Author: Richard Smith > Date: 2020-01-30T17:16:50-08:00 > New Revision:

[PATCH] D73208: [objc_direct] fix codegen for mismatched Decl/Impl return types

dexonsmith accepted this revision. dexonsmith added a comment. This revision is now accepted and ready to land. LGTM, with one style nitpick. Comment at: clang/lib/CodeGen/CGObjCMac.cpp:4049-4051 +if (!OMD->getBody() || COMD->getReturnType() == OMD->getReturnType()) { +

[clang] d28763c - Replace 'AllowExplicit' bool with an enum. No functionality change.

Author: Richard Smith Date: 2020-01-30T17:16:50-08:00 New Revision: d28763cad06e66651e1936756d3e28126b055abf URL: https://github.com/llvm/llvm-project/commit/d28763cad06e66651e1936756d3e28126b055abf DIFF: https://github.com/llvm/llvm-project/commit/d28763cad06e66651e1936756d3e28126b055abf.diff

[clang] 1f3f8c3 - PR44721: Don't consider overloaded operators for built-in comparisons

Author: Richard Smith Date: 2020-01-30T17:16:50-08:00 New Revision: 1f3f8c369a5067a132c871f33a955a7feaea8534 URL: https://github.com/llvm/llvm-project/commit/1f3f8c369a5067a132c871f33a955a7feaea8534 DIFF: https://github.com/llvm/llvm-project/commit/1f3f8c369a5067a132c871f33a955a7feaea8534.diff

[PATCH] D73245: Don't define std::max_align_t if not used in C++03 mode

joerg added a comment. Let me clarify the situation for a moment: (1) libc++ does try to work in C++03 mode. See the separate implementation of for pre-C++11. It is also desirable to support. This is completely beside the question of TR1 support. (2) The only reason why max_align_t is

[PATCH] D73543: [clang] Add support for __builtin_memcpy_inline

efriedma added inline comments. Comment at: clang/lib/Sema/SemaChecking.cpp:1653 +llvm::APSInt I(64); +if (!TheCall->getArg(2)->isIntegerConstantExpr(I, Context)) + Diag(TheCall->getBeginLoc(), diag::err_expr_not_ice) SemaBuiltinConstantArg. Or

[PATCH] D70950: Add ProcName to TimeTraceProfiler

rnk added a comment. This broke ClangBuildAnalyzer on Windows because it has a very naive check for "clang": https://github.com/aras-p/ClangBuildAnalyzer/blob/master/src/main.cpp#L148 I was wondering why this wasn't working on Windows anymore. =/ Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D73282: Fix debug-info generation for block invocations so that we set the LinkageName instead of the Name

dblaikie added inline comments. Comment at: clang/lib/CodeGen/CGDebugInfo.cpp:3659 // Use llvm function name. -Name = Fn->getName(); +if (Fn->getName().startswith("___Z")) + LinkageName = Fn->getName(); shafik wrote: > dblaikie wrote: > >

[PATCH] D73282: Fix debug-info generation for block invocations so that we set the LinkageName instead of the Name

shafik marked an inline comment as done. shafik added inline comments. Comment at: clang/lib/CodeGen/CGDebugInfo.cpp:3659 // Use llvm function name. -Name = Fn->getName(); +if (Fn->getName().startswith("___Z")) + LinkageName = Fn->getName();

[PATCH] D73208: [objc_direct] fix codegen for mismatched Decl/Impl return types

MadCoder updated this revision to Diff 241599. MadCoder added a comment. @dexonsmith here, I still hook the same method but do it in a more LLVM-approved way ;) It works with minimap perf impect because the only case we call GenerateDirectMethod with an Implementation is if we're about to

[PATCH] D73208: [objc_direct] fix codegen for mismatched Decl/Impl return types

MadCoder updated this revision to Diff 241601. MadCoder added a comment. damn you tabs! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73208/new/ https://reviews.llvm.org/D73208 Files: clang/lib/CodeGen/CGObjCMac.cpp clang/test/CodeGenObjC/direct-method-ret-mismatch.m Index:

[PATCH] D73734: [Fuchsia] Never link in implicit "system dependencies" of sanitizer runtimes

This revision was automatically updated to reflect the committed changes. Closed by commit rGfdf31ae32dd2: [Fuchsia] Never link in implicit system dependencies of sanitizer runtimes (authored by mcgrathr, committed by phosek). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] fdf31ae - [Fuchsia] Never link in implicit "system dependencies" of sanitizer runtimes

Author: Roland McGrath Date: 2020-01-30T14:21:23-08:00 New Revision: fdf31ae32dd2bddd9cc72ee7108f0c69bb2e637f URL: https://github.com/llvm/llvm-project/commit/fdf31ae32dd2bddd9cc72ee7108f0c69bb2e637f DIFF:

[PATCH] D73742: [Clang][Driver] After default -fintegrated-cc1, fix report_fatal_error no longer generates preprocessed source + reproducer.sh

aganea created this revision. aganea added reviewers: arichardson, rnk, hans. Herald added subscribers: llvm-commits, cfe-commits, hiraditya. Herald added projects: clang, LLVM. As reported by @arichardson , this patch now makes `llvm::report_fatal_error()` to generate a preprocessed source +

[PATCH] D72876: Create a clang-tidy check to warn when -dealloc is implemented inside an ObjC class category.

mwyman added inline comments. Comment at: clang-tools-extra/clang-tidy/objc/DeallocInCategoryCheck.cpp:34 + +void DeallocInCategoryCheck::check(const MatchFinder::MatchResult ) { + const auto *MatchedDecl = Result.Nodes.getNodeAs("dealloc"); stephanemoore

[PATCH] D72876: Create a clang-tidy check to warn when -dealloc is implemented inside an ObjC class category.

mwyman updated this revision to Diff 241573. mwyman marked 9 inline comments as done. mwyman added a comment. Implemented review feedback. Updated diagnostic message to mention the category name. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D73363: Verify that clang's max alignment is <= LLVM's max alignment

rnk added a comment. I moved this include in rG01943a59f51d8b5ede062305941c1f864b8a6a13 . I meant to paste this in the message, but I'll put it here, since the results were significant: $ diff -u deps-before.txt

[PATCH] D73687: [AArch64][SVE] Add SVE2 intrinsics for complex integer dot product

efriedma added inline comments. Comment at: llvm/include/llvm/IR/IntrinsicsAArch64.td: + LLVMSubdivide4VectorType<0>, + llvm_i32_ty], +[IntrNoMem]>; Missing ImmArg? Comment at:

[clang] 01943a5 - Move verification of Sema::MaximumAlignment to a .cpp file

Author: Reid Kleckner Date: 2020-01-30T13:37:52-08:00 New Revision: 01943a59f51d8b5ede062305941c1f864b8a6a13 URL: https://github.com/llvm/llvm-project/commit/01943a59f51d8b5ede062305941c1f864b8a6a13 DIFF: https://github.com/llvm/llvm-project/commit/01943a59f51d8b5ede062305941c1f864b8a6a13.diff

[PATCH] D73363: Verify that clang's max alignment is <= LLVM's max alignment

rnk added inline comments. Comment at: clang/lib/CodeGen/CGValue.h:18 #include "clang/AST/ASTContext.h" +#include "clang/Sema/Sema.h" #include "clang/AST/Type.h" This includes Sema.h into every codegen file that uses CGValue.h (most of them). That seems bad

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

njames93 added a comment. I toyed with the idea of checking the access specifier for the dependent base classes, but I'm pretty sure it doesn't matter. If the compile finds 2 look ups in one base class it will pick the later dependancy regardless of whether its private or not. And if the

[PATCH] D73734: [Fuchsia] Never link in implicit "system dependencies" of sanitizer runtimes

mcgrathr updated this revision to Diff 241561. mcgrathr added a comment. style nit Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73734/new/ https://reviews.llvm.org/D73734 Files: clang/lib/Driver/ToolChains/CommonArgs.cpp Index:

[PATCH] D73734: [Fuchsia] Never link in implicit "system dependencies" of sanitizer runtimes

mcgrathr updated this revision to Diff 241560. mcgrathr added a comment. builds now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73734/new/ https://reviews.llvm.org/D73734 Files: clang/lib/Driver/ToolChains/CommonArgs.cpp Index:

[PATCH] D73675: Avoid many std::tie/tuple instantiations in ASTImporter

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62352 tests passed, 1 failed and 839 were skipped. failed: libc++.std/containers/sequences/array/array_creation/to_array.fail.cpp {icon times-circle color=red} clang-tidy: fail. clang-tidy found 0 errors and

[PATCH] D73208: [objc_direct] fix codegen for mismatched Decl/Impl return types

dexonsmith added a comment. In D73208#1837909 , @MadCoder wrote: > In D73208#1836789 , @dexonsmith > wrote: > > > I think we're talking across each other. The idea is check the type when > > generating the

[PATCH] D73738: [Clang][Bundler][NFC] Replace SmallString<...> with StringRef

This revision was automatically updated to reflect the committed changes. Closed by commit rG5be2ca29217a: [Clang][Bundler][NFC] Replace SmallString... with StringRef (authored by sdmitriev). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73738/new/

[PATCH] D73675: Avoid many std::tie/tuple instantiations in ASTImporter

rnk updated this revision to Diff 241552. rnk added a comment. - Rewrite with importChecked, no importSeq Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73675/new/ https://reviews.llvm.org/D73675 Files: clang/lib/AST/ASTImporter.cpp Index:

[clang] 5be2ca2 - [Clang][Bundler][NFC] Replace SmallString<...> with StringRef

Author: Sergey Dmitriev Date: 2020-01-30T12:50:04-08:00 New Revision: 5be2ca29217ad977f2479bfc530127c7fb418963 URL: https://github.com/llvm/llvm-project/commit/5be2ca29217ad977f2479bfc530127c7fb418963 DIFF:

[PATCH] D73738: [Clang][Bundler][NFC] Replace SmallString<...> with StringRef

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

[PATCH] D73738: [Clang][Bundler][NFC] Replace SmallString<...> with StringRef

sdmitriev created this revision. sdmitriev added a reviewer: ABataev. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D73738 Files: clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp Index:

[PATCH] D73282: Fix debug-info generation for block invocations so that we set the LinkageName instead of the Name

dblaikie added inline comments. Comment at: clang/lib/CodeGen/CGDebugInfo.cpp:3659 // Use llvm function name. -Name = Fn->getName(); +if (Fn->getName().startswith("___Z")) + LinkageName = Fn->getName(); shafik wrote: > dblaikie wrote: > >

[PATCH] D73354: clang-format: insert trailing commas into containers.

MyDeveloperDay added a comment. Nit: don't forget the ClangFormatStyleOption.rst and ReleaseNote.rst changes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73354/new/ https://reviews.llvm.org/D73354

[PATCH] D73354: clang-format: insert trailing commas into containers.

MyDeveloperDay accepted this revision. MyDeveloperDay added a comment. I'm happy if you are, I think given that you've really been one of the main contributors of clang-format [JS] for the last number of years you are the best person to probably assess the impact on Javascript code. Thanks for

[PATCH] D73734: [Fuchsia] Never link in implicit "system dependencies" of sanitizer runtimes

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM % nit Comment at: clang/lib/Driver/ToolChains/CommonArgs.cpp:586 ArgStringList ) { + if (Triple.isOSFuchsia()) { +// Fuchsia

[PATCH] D73734: [Fuchsia] Never link in implicit "system dependencies" of sanitizer runtimes

mcgrathr created this revision. mcgrathr added a reviewer: phosek. Herald added a project: clang. Herald added a subscriber: cfe-commits. This is never appropriate on Fuchsia and any future needs for system library dependencies of compiler-supplied runtimes will be addressed via `.deplibs`

[PATCH] D54943: [clang-tidy] implement const-transformation for cppcoreguidelines-const-correctness

JonasToth added a comment. In D54943#1841086 , @AlexanderLanin wrote: > In D54943#1815772 , @0x8000- > wrote: > > > This generated 56 "const const" declarations, of which 25 were triple const! > This could

Re: Getting started

Hi Anshil, take a look at this: https://clang.llvm.org/docs/InternalsManual.html cxx1x is probably done already, but cxx2a has some outstanding features ;) In general, please use 'cfe-dev' instead of 'cfe-commits'. There is the normal discussion and questions/answers for developers. Regards

[PATCH] D73675: Avoid many std::tie/tuple instantiations in ASTImporter

rnk planned changes to this revision. rnk marked an inline comment as done. rnk added a comment. In D73675#1849182 , @martong wrote: > Thanks for this patch too! > However, I have a minor concern with this new approach: > If `importSeq` fails then a

[PATCH] D73242: [WPD/LowerTypeTests] Delay lowering/removal of type tests until after ICP

tejohnson added a comment. In D73242#1849484 , @tejohnson wrote: > Both of these approaches need the type tests to determine the correct address > point offset (the offset in the type test) to use in the compare sequence. I typed this too fast - the

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62347 tests passed, 1 failed and 839 were skipped. failed: libc++.std/containers/sequences/array/array_creation/to_array.fail.cpp {icon check-circle color=green} clang-tidy: pass. {icon check-circle

[PATCH] D73667: Speed up compilation of ASTImporter

This revision was automatically updated to reflect the committed changes. Closed by commit rGaf3e88495627: Speed up compilation of ASTImporter (authored by rnk). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73667/new/

[clang] af3e884 - Speed up compilation of ASTImporter

Author: Reid Kleckner Date: 2020-01-30T11:01:24-08:00 New Revision: af3e88495627c9b150814ff13e5749e1ed31c5d3 URL: https://github.com/llvm/llvm-project/commit/af3e88495627c9b150814ff13e5749e1ed31c5d3 DIFF: https://github.com/llvm/llvm-project/commit/af3e88495627c9b150814ff13e5749e1ed31c5d3.diff

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

njames93 updated this revision to Diff 241515. njames93 added a comment. - Streamline memberExpr matchers Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73052/new/ https://reviews.llvm.org/D73052 Files:

[clang] 60f5da7 - [Concepts] Add 'this' context to instantiation of member requires clause

Author: Saar Raz Date: 2020-01-30T20:47:59+02:00 New Revision: 60f5da79e3de49b2074446e656a72970499a8d78 URL: https://github.com/llvm/llvm-project/commit/60f5da79e3de49b2074446e656a72970499a8d78 DIFF: https://github.com/llvm/llvm-project/commit/60f5da79e3de49b2074446e656a72970499a8d78.diff

[clang] a424ef9 - [Concepts] Add check for dependent RC when checking function constraints

Author: Saar Raz Date: 2020-01-30T20:46:32+02:00 New Revision: a424ef99e7b9821ec80564af3d3a8f091323a38c URL: https://github.com/llvm/llvm-project/commit/a424ef99e7b9821ec80564af3d3a8f091323a38c DIFF: https://github.com/llvm/llvm-project/commit/a424ef99e7b9821ec80564af3d3a8f091323a38c.diff

[clang] c83d9be - [Concept] Fix incorrect check for containsUnexpandedParameterPack in CSE

Author: Saar Raz Date: 2020-01-30T20:45:44+02:00 New Revision: c83d9bedc0cc430dc620e7a807daeb985d390325 URL: https://github.com/llvm/llvm-project/commit/c83d9bedc0cc430dc620e7a807daeb985d390325 DIFF: https://github.com/llvm/llvm-project/commit/c83d9bedc0cc430dc620e7a807daeb985d390325.diff

[PATCH] D73729: [analyzer] AnalyzerOptions: Remove 'fixits-as-remarks'

Charusso created this revision. Charusso added a reviewer: NoQ. Charusso added a project: clang. Herald added subscribers: cfe-commits, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun. Charusso added a parent revision: D69746: [analyzer] FixItHint:

[PATCH] D73282: Fix debug-info generation for block invocations so that we set the LinkageName instead of the Name

shafik updated this revision to Diff 241508. shafik marked 4 inline comments as done. shafik added a comment. Address comments by not special casing the `___Z` case, this requires setting both the Name and Linkage name because DWARF requires subroutines to have a `DW_AT_name`. We discovered

[PATCH] D73282: Fix debug-info generation for block invocations so that we set the LinkageName instead of the Name

shafik marked an inline comment as done. shafik added inline comments. Comment at: clang/lib/CodeGen/CGDebugInfo.cpp:3659 // Use llvm function name. -Name = Fn->getName(); +if (Fn->getName().startswith("___Z")) + LinkageName = Fn->getName();

[PATCH] D73721: [Clang][Driver] Disable llvm passes for the first host OpenMP offload compilation

This revision was automatically updated to reflect the committed changes. Closed by commit rG36bfdb7096cf: [Clang][Driver] Disable llvm passes for the first host OpenMP offload… (authored by sdmitriev). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] 36bfdb7 - [Clang][Driver] Disable llvm passes for the first host OpenMP offload compilation

Author: Sergey Dmitriev Date: 2020-01-30T10:16:41-08:00 New Revision: 36bfdb7096cfe1925448e408ec72f1a6bdc4cd2c URL: https://github.com/llvm/llvm-project/commit/36bfdb7096cfe1925448e408ec72f1a6bdc4cd2c DIFF:

[PATCH] D70172: [CUDA][HIP][OpenMP] Emit deferred diagnostics by a post-parsing AST travese

rjmccall added inline comments. Comment at: clang/lib/Sema/SemaExpr.cpp:17183 + class DeferredDiagnosticsEmitter + : public EvaluatedExprVisitor { +Sema Is there any way to share most of the visitation logic here with the visitor we use in

[PATCH] D73721: [Clang][Driver] Disable llvm passes for the first host OpenMP offload compilation

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

[PATCH] D73723: [clangd][Hover] Handle uninstantiated default args

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62330 tests passed, 0 failed and 838 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon check-circle color=green} clang-format: pass. Build artifacts

[PATCH] D73360: [OpenCL] Restrict address space conversions in nested pointers

rjmccall added a comment. Is there no follow-up code when actually emitting the failure diagnostic which tries to figure out a more specific cause of failure? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73360/new/ https://reviews.llvm.org/D73360

[PATCH] D73723: [clangd][Hover] Handle uninstantiated default args

kadircet created this revision. kadircet added reviewers: sammccall, ilya-biryukov. Herald added subscribers: cfe-commits, usaxena95, arphaman, jkorous, MaskRay. Herald added a project: clang. Default args might exist but be unparsed or uninstantiated. getDefaultArg asserts on those. This patch

[PATCH] D69746: [analyzer] FixItHint: Apply and test hints with the Clang Tidy's script

Charusso updated this revision to Diff 241492. Charusso marked 8 inline comments as done. Charusso added a comment. - Change to 4-column space standard. - Simplify obtaining the Clang include directory. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69746/new/

[PATCH] D67399: [ARM] Follow AACPS for preserving number of loads/stores of volatile bit-fields

jfb accepted this revision. jfb added a comment. This revision is now accepted and ready to land. I'm happy with this change since it's opt-in. Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67399/new/ https://reviews.llvm.org/D67399

[PATCH] D69746: [analyzer] FixItHint: Apply and test hints with the Clang Tidy's script

Charusso updated this revision to Diff 241493. Charusso edited the summary of this revision. Charusso added a comment. - Rename the script from `check_analyzer_fixit.py` to `check-analyzer-fixit.py` CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69746/new/

[PATCH] D69746: [analyzer] FixItHint: Apply and test hints with the Clang Tidy's script

Charusso added a comment. Thanks for the reviews! Sorry for the delay. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69746/new/ https://reviews.llvm.org/D69746 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D69746: [analyzer] FixItHint: Apply and test hints with the Clang Tidy's script

Charusso added inline comments. Comment at: clang/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp:92 + + bool IncludePath = false, ShouldEmitAsError = false, FixitsAsRemarks = false, + ApplyFixIts = false; NoQ wrote: > I'll be perfectly happy with

[clang-tools-extra] 3ae11b4 - [NFC] small refactor on RenamerClangTidyCheck.cpp

Author: Nathan James Date: 2020-01-30T17:32:06Z New Revision: 3ae11b42818363f70b3c6b0246bb617e35709c58 URL: https://github.com/llvm/llvm-project/commit/3ae11b42818363f70b3c6b0246bb617e35709c58 DIFF: https://github.com/llvm/llvm-project/commit/3ae11b42818363f70b3c6b0246bb617e35709c58.diff LOG:

[PATCH] D73721: [Clang][Driver] Disable llvm passes for the first host OpenMP offload compilation

sdmitriev created this revision. sdmitriev added reviewers: ABataev, jdoerfert, hfinkel. Herald added subscribers: cfe-commits, guansong. Herald added a project: clang. With OpenMP offloading host compilation is done in two phases to capture host IR that is passed to all device compilations as

[PATCH] D73701: [clang] fix linkage of nested lambda

hliao added a comment. + rsmith Comment at: clang/lib/AST/Decl.cpp:1416-1417 return getLVForClosure( - OuterMostLambda->getDeclContext()->getRedeclContext(), - OuterMostLambda->getLambdaContextDecl(), computation); +

[PATCH] D73720: [Analyzer] Use note tags to track container begin and and changes

baloghadamsoftware created this revision. baloghadamsoftware added reviewers: NoQ, Szelethus. baloghadamsoftware added a project: clang. Herald added subscribers: Charusso, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, rnkovacs, szepet, xazax.hun. Container operations such as `push_back()`,

[PATCH] D72811: [WIP][OPENMP5.0] allow lvalue for motion clause

ABataev added inline comments. Comment at: clang/lib/Sema/SemaOpenMP.cpp:15201 +namespace { +class LocatorChecker final : public StmtVisitor { + OMPClauseMappableExprCommon::MappableExprComponentList Components; Seems to me, you're skipping many expressions,

[PATCH] D73637: Fix handling of OO_Spaceship in DecodeOperatorCall

sberg updated this revision to Diff 241485. sberg added a comment. Minimal reproducer is template struct S; template void operator <=>(S, S); template void f(T x) { decltype(x <=> x) v; } Added a corresponding test to clang/test/SemaCXX/cxx2a-three-way-comparison.cpp. CHANGES SINCE LAST

[PATCH] D67399: [ARM] Follow AACPS for preserving number of loads/stores of volatile bit-fields

dnsampaio updated this revision to Diff 241486. dnsampaio added a comment. - Added flag to allow user to opt-in Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67399/new/ https://reviews.llvm.org/D67399 Files:

[clang] 4697874 - [OPENMP50]Handle lastprivate conditionals passed as shared in inner

Author: Alexey Bataev Date: 2020-01-30T11:35:23-05:00 New Revision: 4697874c28eda11ce29266f3c6188280809b6b36 URL: https://github.com/llvm/llvm-project/commit/4697874c28eda11ce29266f3c6188280809b6b36 DIFF: https://github.com/llvm/llvm-project/commit/4697874c28eda11ce29266f3c6188280809b6b36.diff

[PATCH] D73462: [dwarf-5] Support DebugInfo for Defaulted parameters for C++ templates

aprantl added a comment. In addition to the roundtrip test, can you please also add a .bc file test containing a DITemplateTypeParameter / DITemplateValueParameter produced by today's llvm to test the bitcode upgrade code added to MetadataLoader? You'll find examples if you search for .bc

[PATCH] D73719: [AArch64][SVE] Add SVE2 intrinsics for widening DSP operations

kmclaughlin created this revision. kmclaughlin added reviewers: sdesmalen, dancgr, efriedma, cameron.mcinally. Herald added subscribers: psnobl, rkruppe, hiraditya, kristof.beyls, tschuett. Herald added a reviewer: rengolin. Herald added a project: LLVM. Implements the following intrinsics: -

[PATCH] D72705: [clang][checkers] Added new checker 'alpha.unix.ErrorReturn'.

balazske updated this revision to Diff 241478. balazske added a comment. Adding a new implementation. Now the conditions itself are checked, not the constraints. The appearance of the value to check is detected and tested if it is inside an error check-looking construct. The location of the

[PATCH] D70172: [CUDA][HIP][OpenMP] Emit deferred diagnostics by a post-parsing AST travese

yaxunl added a comment. In D70172#1812749 , @rjmccall wrote: > In D70172#1812664 , @yaxunl wrote: > > > In D70172#1812631 , @rjmccall > > wrote: > > > > > Most uses of the

[PATCH] D72932: [ARM] Follow AACPS standard for volatile bit-fields access width

dnsampaio updated this revision to Diff 241477. dnsampaio added a comment. - Do not generate special volatile access if the record alignment is smaller than the bit-field declared type Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72932/new/

[PATCH] D73642: [Clang][Bundler] Reduce fat object size

This revision was automatically updated to reflect the committed changes. Closed by commit rGc53cb2bdc78e: [Clang][Bundler] Reduce fat object size (authored by sdmitriev). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73642/new/

[PATCH] D73715: - Update .clang-tidy to ignore parameters of main like functions for naming violations in clang and llvm directory

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62329 tests passed, 0 failed and 838 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon check-circle color=green} clang-format: pass. Build artifacts

[clang] c53cb2b - [Clang][Bundler] Reduce fat object size

Author: Sergey Dmitriev Date: 2020-01-30T08:21:39-08:00 New Revision: c53cb2bdc78ee3d3eec8c30821480b390b0eb145 URL: https://github.com/llvm/llvm-project/commit/c53cb2bdc78ee3d3eec8c30821480b390b0eb145 DIFF:

[PATCH] D73715: - Update .clang-tidy to ignore parameters of main like functions for naming violations in clang and llvm directory

njames93 created this revision. njames93 added reviewers: alexfh, hokein. Herald added subscribers: llvm-commits, cfe-commits, aheejin. Herald added projects: clang, LLVM. njames93 marked an inline comment as done. njames93 added inline comments. Comment at: clang/.clang-tidy:23

[PATCH] D69731: [analyzer] CheckerContext: Make the Preprocessor available

This revision was automatically updated to reflect the committed changes. Closed by commit rG38ab3b876baa: [analyzer] CheckerContext: Make the Preprocessor available (authored by Charusso). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69731/new/

[PATCH] D73715: - Update .clang-tidy to ignore parameters of main like functions for naming violations in clang and llvm directory

njames93 marked an inline comment as done. njames93 added inline comments. Comment at: clang/.clang-tidy:23 + - key: readability-identifier-naming.IgnoreMainLikeFunctions +value: 1 Although identifier-naming is disabled, its still a

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62329 tests passed, 0 failed and 838 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon check-circle color=green} clang-format: pass. Build artifacts

[clang] 38ab3b8 - [analyzer] CheckerContext: Make the Preprocessor available

Author: Charusso Date: 2020-01-30T17:05:52+01:00 New Revision: 38ab3b876baaa899b92dda9113a4d1d4b56c2e79 URL: https://github.com/llvm/llvm-project/commit/38ab3b876baaa899b92dda9113a4d1d4b56c2e79 DIFF: https://github.com/llvm/llvm-project/commit/38ab3b876baaa899b92dda9113a4d1d4b56c2e79.diff

  1   2   >