[PATCH] D71460: [OpenCL] Fix support for cl_khr_mipmap_image_writes

AlexeySotkin updated this revision to Diff 234674. AlexeySotkin added a comment. Rename `color` to `depth` CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71460/new/ https://reviews.llvm.org/D71460 Files: clang/include/clang/Basic/OpenCLExtensions.def clang/lib/Headers/opencl-c.h c

[PATCH] D70926: [clang-format] Add option for not breaking line before ObjC params

MyDeveloperDay accepted this revision. MyDeveloperDay added a comment. This revision is now accepted and ready to land. Thanks for the patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70926/new/ https://reviews.llvm.org/D70926 __

[PATCH] D71694: [objc_direct] Tigthen checks for direct methods

MadCoder created this revision. MadCoder added reviewers: rjmccall, dexonsmith, erik.pilkington, ahatanak, arphaman. MadCoder added a project: clang. Herald added a subscriber: cfe-commits. Because the name of a direct method must be agreed upon by the caller and the implementation, certain bad

[PATCH] D71650: [AST] Fix compilation with GCC < 8 for MinGW

This revision was automatically updated to reflect the committed changes. Closed by commit rGf20fc65887e2: [clang] Fix compilation with GCC < 8 for MinGW (authored by mstorsjo). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71650/new/ https://review

[PATCH] D71694: [objc_direct] Tigthen checks for direct methods

MadCoder updated this revision to Diff 234684. MadCoder added a comment. Small update because I uploaded before the full completion of the test-suite and somehow it's sentient and punished me... (some diagnostics were too verbose and stuttering things with `Sema::CheckObjCMethodOverrides`). CH

[PATCH] D71694: [objc_direct] Tigthen checks for direct methods

MadCoder added a subscriber: aprantl. MadCoder added inline comments. Comment at: clang/test/CodeGenObjC/direct-method.m:196 +@implementation Foo (Cat) +// CHECK-LABEL: define hidden i32 @"\01-[Foo(Cat) directMethodInCategory]"( +- (int)directMethodInCategory { t

[clang-tools-extra] ac3f9e4 - [clangd] Improve documentation for auto and implicit specs

Author: Kadir Cetinkaya Date: 2019-12-19T11:55:22+01:00 New Revision: ac3f9e48421712168884d59cbfe8b294dd76a19b URL: https://github.com/llvm/llvm-project/commit/ac3f9e48421712168884d59cbfe8b294dd76a19b DIFF: https://github.com/llvm/llvm-project/commit/ac3f9e48421712168884d59cbfe8b294dd76a19b.dif

[PATCH] D71596: [clangd] Improve documentation for auto and implicit specs

This revision was automatically updated to reflect the committed changes. Closed by commit rGac3f9e484217: [clangd] Improve documentation for auto and implicit specs (authored by kadircet). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71596/new/ ht

[PATCH] D71533: [clangd] Show template arguments in type hierarchy when possible

kadircet added inline comments. Comment at: clang-tools-extra/clangd/XRefs.cpp:687 - const Decl *D = Decls[0]; + const Decl *D = nullptr; + for (const Decl *Candidate : Decls) { kadircet wrote: > maybe just > ``` > const Decl *D = Decls.front() > for(const a

[PATCH] D70157: Align branches within 32-Byte boundary(NOP padding)

skan updated this revision to Diff 234697. skan added a comment. Add more comment CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70157/new/ https://reviews.llvm.org/D70157 Files: llvm/include/llvm/MC/MCAsmBackend.h llvm/include/llvm/MC/MCAssembler.h llvm/include/llvm/MC/MCFragment

[PATCH] D71556: [AArch64][SVE] Implement intrinsic for non-faulting loads

kmclaughlin planned changes to this revision. kmclaughlin added a comment. Thanks for the feedback on this patch, @efriedma & @sdesmalen! I think there is still value in adding a NonFaulting flag to MachineMemOperand so that we can benefit from legalisation, but as this is not a requirement for

[PATCH] D71698: [AArch64][SVE] Add intrinsic for non-faulting loads

kmclaughlin created this revision. kmclaughlin added reviewers: sdesmalen, efriedma, andwar, dancgr, mgudim. Herald added subscribers: psnobl, rkruppe, hiraditya, kristof.beyls, tschuett. Herald added a reviewer: rengolin. Herald added a project: LLVM. This patch adds the llvm.aarch64.sve.ldnf1 in

[clang] dde7b6b - Re-land "Add an -fno-temp-file flag for compilation"

Author: Hans Wennborg Date: 2019-12-19T13:34:52+01:00 New Revision: dde7b6bcda9d8cf874fd77b98d0d62d6aeaed049 URL: https://github.com/llvm/llvm-project/commit/dde7b6bcda9d8cf874fd77b98d0d62d6aeaed049 DIFF: https://github.com/llvm/llvm-project/commit/dde7b6bcda9d8cf874fd77b98d0d62d6aeaed049.diff

[PATCH] D70615: Add an -fno-temp-file flag for compilation

This revision was automatically updated to reflect the committed changes. Closed by commit rGdde7b6bcda9d: Re-land "Add an -fno-temp-file flag for compilation" (authored by hans). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70615/new/ https://revi

[PATCH] D71686: Fix false positive in magic number checker

Eugene.Zelenko added a comment. Please mention fix in Release Notes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71686/new/ https://reviews.llvm.org/D71686 ___ cfe-commits mailing list cfe-commits@li

[PATCH] D71598: [clangd] Filter implicit references from index while renaming

kbobyrev updated this revision to Diff 234720. kbobyrev added a comment. Move helper function back to the anonymous namespace. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71598/new/ https://reviews.llvm.org/D71598 Files: clang-tools-extra/clangd/refactor/Rename.cpp clang-tools-ex

[PATCH] D71598: [clangd] Filter implicit references from index while renaming

kbobyrev updated this revision to Diff 234719. kbobyrev marked 7 inline comments as done. kbobyrev added a comment. Sorry for a delay: I was trying to work with range patching heuristics and get it to work in generic case of "stale index returns more results than lexer", but in the end I converg

[PATCH] D71707: clang-tidy: new bugprone-pointer-cast-widening

jankratochvil created this revision. jankratochvil added reviewers: labath, alexfh, lebedev.ri. jankratochvil added a project: clang-tools-extra. Herald added subscribers: cfe-commits, kristof.beyls, mgorny. Herald added a project: clang. Check for cast of a pointer to wider (even unsigned) intege

[PATCH] D71707: clang-tidy: new bugprone-pointer-cast-widening

jankratochvil added a comment. Example output: PATH="$PWD/bin:$PATH" ~/redhat/llvm-monorepo/clang-tools-extra/clang-tidy/tool/run-clang-tidy.py -checks='-*,bugprone-pointer-cast-widening' /home/jkratoch/redhat/llvm-monorepo/lldb/source/ Enabled checks: bugprone-pointer-cast-wideni

[PATCH] D71598: [clangd] Filter implicit references from index while renaming

kadircet added inline comments. Comment at: clang-tools-extra/clangd/refactor/Rename.cpp:431 +// REQUIRES: Indexed and Lexed are sorted. +// REQUIRES: Indexed and Lexed are sorted. +llvm::Optional> filterIndexResults(ArrayRef Indexed, duplication ==

[PATCH] D71708: [OPENMP50]Codegen for nontemporal clause.

ABataev created this revision. ABataev added reviewers: rjmccall, hfinkel. Herald added subscribers: arphaman, guansong. Herald added a reviewer: jdoerfert. Herald added a project: clang. Basic codegen for the declarations marked as nontemporal. Also, if the base declaration in the member expressi

[PATCH] D71709: Give frontend dump flags consistent names (*-dump instead of dump-*)

herzka created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. herzka added a reviewer: modocache. herzka added a comment. @modocache, should I add a unit test for these aliases or would that be too trivial? I don't see unit tests for all the existing flags.

[PATCH] D71709: Give frontend dump flags consistent names (*-dump instead of dump-*)

herzka added a comment. @modocache, should I add a unit test for these aliases or would that be too trivial? I don't see unit tests for all the existing flags. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71709/new/ https://reviews.llvm.org/D7170

[PATCH] D71707: clang-tidy: new bugprone-pointer-cast-widening

labath resigned from this revision. labath added a comment. Though this *was* my idea, I don't really feel qualified to review code here. However, some things to consider: - disallowing casts to intptr_t seems too restrictive -- I doubt many people are doing that, but I guess this type exists f

[PATCH] D71598: [clangd] Filter implicit references from index while renaming

kbobyrev updated this revision to Diff 234727. kbobyrev marked 5 inline comments as done. kbobyrev added a comment. Addressed a bunch of comments to cleanup the patch and replied to ask for clarification of several unresolved comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71598

[PATCH] D71598: [clangd] Filter implicit references from index while renaming

kbobyrev added inline comments. Comment at: clang-tools-extra/clangd/refactor/Rename.cpp:591 // Details: -// - lex the draft code to get all rename candidates, this yields a superset -//of candidates. +// - lex the draft code to get all rename candidates, this yields a set

[PATCH] D71708: [OPENMP50]Codegen for nontemporal clause.

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 61027 tests passed, 0 failed and 728 were skipped. {icon times-circle color=red} clang-tidy: fail. Please fix clang-tidy findings

[PATCH] D71572: [ItaniumCXXABI] Make tls wrappers comdat on Windows

mstorsjo added a comment. In D71572#1788786 , @rnk wrote: > Looks like @rsmith did this here: > > https://github.com/llvm/llvm-project/commit/fbe2369f1a514423e4c25417ab3532502fde6f2a > > I see that it was replacing a CHECK for a specific comdat group wit

[PATCH] D71510: [clang][checkers] Added new checker 'error-return-checker'. (WIP)

balazske updated this revision to Diff 234730. balazske added a comment. Adding a new diff over the previous one. (The commit was amended accidentally.) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71510/new/ https://reviews.llvm.org/D71510 Files

[PATCH] D71451: Support to emit extern variables debuginfo with "-fstandalone-debug"

Jac1494 added a comment. >> @Jac1494 - have you made any measurements of the size increase of this >> change? Perhaps a self-host build of clang? With change(default) and without change size diffrence given below ,in that only debug section size is changed. Without change:-(default) $size -A -

[clang] 2520bef - [Clang FE, SystemZ] Recognize -mrecord-mcount CL option.

Author: Jonas Paulsson Date: 2019-12-19T08:51:55-08:00 New Revision: 2520bef865329d4c04e2de30c222ad0d5ad13ccc URL: https://github.com/llvm/llvm-project/commit/2520bef865329d4c04e2de30c222ad0d5ad13ccc DIFF: https://github.com/llvm/llvm-project/commit/2520bef865329d4c04e2de30c222ad0d5ad13ccc.diff

[PATCH] D70157: Align branches within 32-Byte boundary(NOP padding)

reames added a comment. In D70157#1789159 , @skan wrote: > In D70157#1788445 , @reames wrote: > > > Specifically on the revised patch, I remain confused by the need for > > multiple subtypes. The need for fragment

[PATCH] D71707: clang-tidy: new bugprone-pointer-cast-widening

Eugene.Zelenko added a comment. Please add documentation and mention new check in Release Notes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71707/new/ https://reviews.llvm.org/D71707 ___ cfe-commits

[PATCH] D71714: [Sema] NFC: Remove trailing spaces and fix a typo in a test file

ilya created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. [Sema] Fix -Warray-bounds false negative when casting an out-of-bounds array item Fixes: http://llvm.org/PR44343 Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D71714 Files: c

[PATCH] D71707: clang-tidy: new bugprone-pointer-cast-widening

jankratochvil added a subscriber: labath. jankratochvil added a comment. In D71707#1791280 , @labath wrote: > - disallowing casts to intptr_t seems too restrictive -- I doubt many people > are doing that, but I guess this type exists for a reason, and sin

[PATCH] D71314: Emit a warning if a variable is uninitialized in indirect ASM goto destination.

nickdesaulniers added a comment. > Use -Wuninitialized. D'oh! Comment at: clang/lib/Analysis/UninitializedValues.cpp:856 if (!as->isAsmGoto()) return; nickdesaulniers wrote: > Should we mark the vals `MayUninitialized` here, rather than below? Then I

[PATCH] D71716: [ItaniumCXXABI] Don't mark an extern_weak init function as dso_local on windows

mstorsjo created this revision. mstorsjo added reviewers: rnk, smeenai, rsmith. Herald added a project: clang. Since 6bf108d77a3c, we try to not mark extern_weak symbols as dso_local, to allow using COFF stubs for references to those symbols (as the symbol may be missing, resolving to an absolut

[PATCH] D71314: Emit a warning if a variable is uninitialized in indirect ASM goto destination.

nickdesaulniers added inline comments. Comment at: clang/lib/Analysis/UninitializedValues.cpp:856 if (!as->isAsmGoto()) return; nickdesaulniers wrote: > nickdesaulniers wrote: > > Should we mark the vals `MayUninitialized` here, rather than below? Then

[PATCH] D71714: [Sema] Fix -Warray-bounds false negative when casting an out-of-bounds array item

riccibruno added a comment. These are not the only AST nodes representing cast expressions (there is also `CXXFunctionalCastExpr`, ...). What about replacing the `IgnoreParenImpCasts()` above by `IgnoreParenCasts()` ? Incidentally doing this uncovers another test (`Parser/cxx-ambig-decl-expr.cp

[PATCH] D71709: Give frontend dump flags consistent names (*-dump instead of dump-*)

modocache requested changes to this revision. modocache added subscribers: jroelofs, echristo. modocache added a comment. This revision now requires changes to proceed. Grepping for "dump-tokens", I can see one regression test that exercises this option: clang/test/Lexer/dollar-idents.c. "dump-co

[PATCH] D71709: Give frontend dump flags consistent names (*-dump instead of dump-*)

herzka updated this revision to Diff 234746. herzka added a comment. Update argument parsing Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71709/new/ https://reviews.llvm.org/D71709 Files: clang/include/clang/Driver/CC1Options.td clang/lib/Fro

[PATCH] D69876: Allow output constraints on "asm goto"

nickdesaulniers requested changes to this revision. nickdesaulniers added a comment. This revision now requires changes to proceed. Thinking hard about this, I think there's four cases we really need to think hard about: 1. asm redirects control flow BEFORE assigning to output. ie. int foo(vo

[PATCH] D71718: [X86] Mark various pointer arguments in builtins as const

wristow created this revision. wristow added a reviewer: craig.topper. Enabling `-Wcast-qual` identified many casts in various system headers that were dropping the `const` qualifier. Fixing those missing qualifiers pointed out that a few of the definitions of the builtins did not properly identi

[PATCH] D71718: [X86] Mark various pointer arguments in builtins as const

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

[PATCH] D71451: Support to emit extern variables debuginfo with "-fstandalone-debug"

dblaikie added a comment. In D71451#1791314 , @Jac1494 wrote: > >> @Jac1494 - have you made any measurements of the size increase of this > >> change? Perhaps a self-host build of clang? > > With change(default) and without change size diffrence given bel

[clang] 6a7df3a - [ASTImporter][LLDB] Modifying ImportDeclContext(...) to ensure that we complete each FieldDecl of a RecordDecl when we are importing the definiton

Author: shafik Date: 2019-12-19T11:16:54-08:00 New Revision: 6a7df3a3f940473088b1db1ccadafe52bb274b62 URL: https://github.com/llvm/llvm-project/commit/6a7df3a3f940473088b1db1ccadafe52bb274b62 DIFF: https://github.com/llvm/llvm-project/commit/6a7df3a3f940473088b1db1ccadafe52bb274b62.diff LOG: [

[PATCH] D71378: Modifying ImportDeclContext(...) to ensure that we complete each FieldDecl of a RecordDecl when we are importing the definiton

This revision was automatically updated to reflect the committed changes. Closed by commit rG6a7df3a3f940: [ASTImporter][LLDB] Modifying ImportDeclContext(...) to ensure that we complete… (authored by shafik). Herald added projects: clang, LLDB. Repository: rG LLVM Github Monorepo CHANGES SINC

[PATCH] D71623: [compiler-rt] [test] Add missing %run to fread_fwrite MSAN test

This revision was automatically updated to reflect the committed changes. Closed by commit rG20345707b2c9: [compiler-rt] [test] Add missing %run to fread_fwrite MSAN test (authored by mgorny). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71623/new/

[PATCH] D61634: [clang/llvm] Allow efficient implementation of libc's memory functions in C/C++

jdoerfert resigned from this revision. jdoerfert added a comment. This has enough active reviewers as it is. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61634/new/ https://reviews.llvm.org/D61634 ___

[clang] 7fcd9e3 - [X86] Mark various pointer arguments in builtins as const

Author: Warren Ristow Date: 2019-12-19T11:42:11-08:00 New Revision: 7fcd9e3f70830a9c4bf631a602c2764180b5c3a8 URL: https://github.com/llvm/llvm-project/commit/7fcd9e3f70830a9c4bf631a602c2764180b5c3a8 DIFF: https://github.com/llvm/llvm-project/commit/7fcd9e3f70830a9c4bf631a602c2764180b5c3a8.diff

[PATCH] D71718: [X86] Mark various pointer arguments in builtins as const

This revision was automatically updated to reflect the committed changes. Closed by commit rG7fcd9e3f7083: [X86] Mark various pointer arguments in builtins as const (authored by wristow). Herald added a project: clang. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://rev

[PATCH] D71248: [clangd] Introduce paragraph, the first part of new rendering structs

aganea added inline comments. Comment at: clang-tools-extra/clangd/FormattedString.cpp:110 +// Separate from next block. +*WritePtr++ = ' '; + } There's an issue at this line, the iterator might go past the end of the string (if running the `Document.Se

[PATCH] D71723: ConstrainedFP: use API compatible with opaque pointers.

t.p.northover created this revision. t.p.northover added a reviewer: kpn. Herald added a subscriber: mcrosier. Herald added projects: clang, LLVM. This just updates an IRBuilder interface to take Functions instead of Values so the type can be derived, and fixes some callsites in Clang to call the

[PATCH] D69876: Allow output constraints on "asm goto"

void added a comment. In D69876#1791475 , @nickdesaulniers wrote: > Thinking hard about this, I think there's four cases we really need to think > hard about: > > 1. asm redirects control flow BEFORE assigning to output. ie. > > ``` int foo(void) { int

[PATCH] D69876: Allow output constraints on "asm goto"

void added a comment. I'm not getting the `Undefined temporary symbol` in your example (even with optimizations): [morbo@fawn:llvm-project] cat ../bug.c void baz(void) { int y; asm volatile goto ("mov 42, %0; ja %1": "=d"(y) ::: quux); asm volatile goto ("mov 42, %0; ja %1"

[PATCH] D71694: [objc_direct] Tigthen checks for direct methods

aprantl added a comment. Added superficial LLVM coding style comments. Comment at: clang/lib/Sema/SemaDeclObjC.cpp:4735 +// Merge directness from the canonical declaration upfront +if (!ObjCMethod->isDirectMethod()) { missing `.` at end of sentence.

[PATCH] D71723: ConstrainedFP: use API compatible with opaque pointers.

kpn accepted this revision. kpn added a comment. This revision is now accepted and ready to land. LGTM. Thanks for the fix! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71723/new/ https://reviews.llvm.org/D71723

[PATCH] D71694: [objc_direct] Tigthen checks for direct methods

aprantl added inline comments. Comment at: clang/lib/AST/DeclObjC.cpp:963 if (ObjCInterfaceDecl *IFD = ImplD->getClassInterface()) { - if (ObjCMethodDecl *MD = IFD->getMethod(getSelector(), - isInstanceMethod())) + // Wh

[clang-tools-extra] 3346cec - [clangd] Fix write past end pointer

Author: Kadir Cetinkaya Date: 2019-12-19T21:50:32+01:00 New Revision: 3346cecd4c0c960377b441606b6382a684daf061 URL: https://github.com/llvm/llvm-project/commit/3346cecd4c0c960377b441606b6382a684daf061 DIFF: https://github.com/llvm/llvm-project/commit/3346cecd4c0c960377b441606b6382a684daf061.dif

[PATCH] D71694: [objc_direct] Tigthen checks for direct methods

MadCoder marked an inline comment as done. MadCoder added inline comments. Comment at: clang/lib/AST/DeclObjC.cpp:963 if (ObjCInterfaceDecl *IFD = ImplD->getClassInterface()) { - if (ObjCMethodDecl *MD = IFD->getMethod(getSelector(), -

[PATCH] D71248: [clangd] Introduce paragraph, the first part of new rendering structs

kadircet marked an inline comment as done. kadircet added inline comments. Comment at: clang-tools-extra/clangd/FormattedString.cpp:110 +// Separate from next block. +*WritePtr++ = ' '; + } aganea wrote: > There's an issue at this line, the iterator migh

[PATCH] D71707: clang-tidy: new bugprone-pointer-cast-widening

jankratochvil updated this revision to Diff 234772. jankratochvil added a comment. Wrote the documentation and Release Notes entry, thanks for the review. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71707/new/ https://reviews.llvm.org/D71707 Fil

[PATCH] D71714: [Sema] Fix -Warray-bounds false negative when casting an out-of-bounds array item

rsmith added a comment. In addition to @riccibruno's comment, I found a couple of other suspicious things nearby (unrelated to the fix in this patch). No need to address them in this patch unless you feel motivated :) Comment at: clang/lib/Sema/SemaChecking.cpp:13384 c

[clang] df2e2ab - Implement latest C++ feature test macro recommendations.

Author: Richard Smith Date: 2019-12-19T12:59:13-08:00 New Revision: df2e2ab07b48b81fb440e3522c6e639e8ef8f2e9 URL: https://github.com/llvm/llvm-project/commit/df2e2ab07b48b81fb440e3522c6e639e8ef8f2e9 DIFF: https://github.com/llvm/llvm-project/commit/df2e2ab07b48b81fb440e3522c6e639e8ef8f2e9.diff

[PATCH] D71619: [CLANG] Alignment specifier not applied to anonymous structure or union

rsmith added a comment. Seems fine to me. @aaron.ballman, want to take a look? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71619/new/ https://reviews.llvm.org/D71619 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.ll

[PATCH] D71725: [OpenCL] Fix inconsistency between opencl and c11 atomic fetch max/min

yaxunl created this revision. yaxunl added reviewers: Anastasia, rjmccall. Herald added a subscriber: jfb. There is some inconsistency between opencl and c11 atomic fetch max/min after https://reviews.llvm.org/D46386 https://reviews.llvm.org/D55562 It is not reasonable to have such inconsistenc

[PATCH] D71726: Let clang atomic builtins fetch add/sub support floating point types

yaxunl created this revision. yaxunl added reviewers: rjmccall, b-sumner. Herald added a subscriber: jfb. Recently atomicrmw started to support fadd/fsub: https://reviews.llvm.org/D53965 However clang atomic builtins fetch add/sub still does not support emitting atomicrmw fadd/fsub. This patch

[PATCH] D71644: [clangd] Heuristically resolve dependent call through smart pointer type

nridge added a comment. Shopping around to other reviewers while Sam's OOO. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71644/new/ https://reviews.llvm.org/D71644 ___ cfe-commits mailing list cfe-com

[clang-tools-extra] 918d393 - Fix some typos in the clang-tools-extra doc

Author: Sylvestre Ledru Date: 2019-12-19T22:23:35+01:00 New Revision: 918d393972237fe2f9c0f4c7cd14ed4ec4ba706a URL: https://github.com/llvm/llvm-project/commit/918d393972237fe2f9c0f4c7cd14ed4ec4ba706a DIFF: https://github.com/llvm/llvm-project/commit/918d393972237fe2f9c0f4c7cd14ed4ec4ba706a.dif

[clang-tools-extra] dac98cf - Fix the links to clang analyzers checkers

Author: Sylvestre Ledru Date: 2019-12-19T22:31:24+01:00 New Revision: dac98cfa03e9c87a57bd47dfb9880fb01999fbff URL: https://github.com/llvm/llvm-project/commit/dac98cfa03e9c87a57bd47dfb9880fb01999fbff DIFF: https://github.com/llvm/llvm-project/commit/dac98cfa03e9c87a57bd47dfb9880fb01999fbff.dif

[PATCH] D71694: [objc_direct] Tigthen checks for direct methods

MadCoder updated this revision to Diff 234777. MadCoder added a comment. Addressed @aprantl review feedback. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71694/new/ https://reviews.llvm.org/D71694 Files: clang/include/clang/Basic/DiagnosticSemaKinds.td clang/lib/AST/DeclObjC.cpp

[PATCH] D71707: clang-tidy: new bugprone-pointer-cast-widening

Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/docs/ReleaseNotes.rst:100 + + Check for cast of a pointer to wider (even unsigned) integer. This will + sign-extend the pointer which happens on 32-bit hosts for 64-bit integers. Checks. One se

[PATCH] D71694: [objc_direct] Tigthen checks for direct methods

MadCoder updated this revision to Diff 234778. MadCoder added a comment. I had forgotten to clang format and left some tabs here... some day I'll fix my vim config CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71694/new/ https://reviews.llvm.org/D71694 Files: clang/include/clang/Bas

[PATCH] D71718: [X86] Mark various pointer arguments in builtins as const

wristow added a comment. Thanks for the quick review. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71718/new/ https://reviews.llvm.org/D71718 ___ cfe-commits mailing list cfe-commits@lists.llvm.org ht

[PATCH] D71619: [CLANG] Alignment specifier not applied to anonymous structure or union

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM! Comment at: clang/test/AST/pr43983.cpp:1 +// RUN: %clang_cc1 -fsyntax-only %s -std=c++11 -ast-dump | FileCheck %s + You can drop the `-st

[PATCH] D71727: [clang][Tooling] Prefer -x over -std when interpolating

kadircet created this revision. kadircet added a reviewer: sammccall. Herald added subscribers: cfe-commits, usaxena95, ilya-biryukov. Herald added a project: clang. Currently interpolation logic prefers -std over -x. But the latter is a more strong signal, so this patch inverts the order and only

[PATCH] D71080: [NFC] Separate getLastArgIntValue to Basic

yaxunl added a comment. ping CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71080/new/ https://reviews.llvm.org/D71080 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D71221: [HIP] Add option --gpu-max-threads-per-block=n

yaxunl added a comment. ping CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71221/new/ https://reviews.llvm.org/D71221 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 85cb560 - ConstrainedFP: use API compatible with opaque pointers.

Author: Tim Northover Date: 2019-12-19T21:50:47Z New Revision: 85cb560b8a421d950ccea593b4ee0569249dc136 URL: https://github.com/llvm/llvm-project/commit/85cb560b8a421d950ccea593b4ee0569249dc136 DIFF: https://github.com/llvm/llvm-project/commit/85cb560b8a421d950ccea593b4ee0569249dc136.diff LOG:

[PATCH] D71723: ConstrainedFP: use API compatible with opaque pointers.

t.p.northover closed this revision. t.p.northover added a comment. Thanks Kevin. Committed as 85cb560b8a4 . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71723/new/ https://revie

[PATCH] D71727: [clang][Tooling] Prefer -x over -std when interpolating

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

[PATCH] D71694: [objc_direct] Tigthen checks for direct methods

aprantl added a subscriber: teemperor. aprantl added inline comments. Comment at: clang/test/CodeGenObjC/direct-method.m:196 +@implementation Foo (Cat) +// CHECK-LABEL: define hidden i32 @"\01-[Foo(Cat) directMethodInCategory]"( +- (int)directMethodInCategory { M

[PATCH] D71221: [HIP] Add option --gpu-max-threads-per-block=n

tra added a comment. What's the use case for this flag? Comment at: clang/lib/CodeGen/TargetInfo.cpp:8067 +unsigned MaxThreadsPerBlock = +IsHIPKernel ? M.getLangOpts().GPUMaxThreadsPerBlock : 256; +std::string AttrVal = std::string("1,") + llvm::utostr(MaxThread

[PATCH] D71694: [objc_direct] Tigthen checks for direct methods

MadCoder marked an inline comment as done. MadCoder added inline comments. Comment at: clang/test/CodeGenObjC/direct-method.m:196 +@implementation Foo (Cat) +// CHECK-LABEL: define hidden i32 @"\01-[Foo(Cat) directMethodInCategory]"( +- (int)directMethodInCategory { -

[PATCH] D71475: [WIP][OPENMP] Try to fix linear clause crash by emitting alloca for step

cchen added a comment. In D71475#1788187 , @ABataev wrote: > The fixed patch. Several codegen tests require some adjustment. @Abataev, thanks for the reply. However, for adjusting the codegen, I found it really hard to update the tests by reading the di

[PATCH] D71533: [clangd] Show template arguments in type hierarchy when possible

nridge updated this revision to Diff 234784. nridge marked 4 inline comments as done. nridge added a comment. Address most review comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71533/new/ https://reviews.llvm.org/D71533 Files: clang-tool

[PATCH] D71533: [clangd] Show template arguments in type hierarchy when possible

nridge marked an inline comment as done. nridge added inline comments. Comment at: clang-tools-extra/clangd/unittests/TypeHierarchyTests.cpp:418 + EXPECT_THAT(*Result, + AllOf(WithName("S<0>"), WithKind(SymbolKind::Struct), Parents())); } kadircet

[PATCH] D71728: [analyzer] Add a syntactic security check for ObjC NSCoder API.

NoQ created this revision. NoQ added a reviewer: dcoughlin. Herald added subscribers: cfe-commits, Charusso, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun. Herald added a project: clang. Method `-[NSCoder decodeValueOfObjCType:at:]` is not only d

[PATCH] D71707: clang-tidy: new bugprone-pointer-cast-widening

jankratochvil updated this revision to Diff 234787. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71707/new/ https://reviews.llvm.org/D71707 Files: clang-tools-extra/clang-tidy/bugprone/BugproneTidyModule.cpp clang-tools-extra/clang-tidy/bugpron

[PATCH] D71728: [analyzer] Add a syntactic security check for ObjC NSCoder API.

dcoughlin accepted this revision. dcoughlin added a comment. This revision is now accepted and ready to land. This looks good to me, but I think we need a deployment target check on the diagnostic since the safe API is only available in iOS 11+, macOS 10.13+, tvOS 11+, and watchOS 4.0+. If the d

[PATCH] D71707: clang-tidy: new bugprone-pointer-cast-widening

jankratochvil marked 3 inline comments as done. jankratochvil added inline comments. Comment at: clang-tools-extra/docs/ReleaseNotes.rst:100 + + Check for cast of a pointer to wider (even unsigned) integer. This will + sign-extend the pointer which happens on 32-bit hosts for 6

LLVM lab network works today

Hello All, LLVM master will be unavailable Thursday, December 19th at 5:30 pm PST due to network maintenance. The planned maintenance duration is 20 minutes but will be performed as soon as possible. Thanks Galina ___ cfe-commits mailing list cfe-commi

[PATCH] D65591: [AST] Add a flag indicating if any subexpression had errors

rsmith added a comment. Summary of an off-line discussion with Ilya: - The current situation where Clang can do delayed typo correction for C++ but not for C is not a good long-term position. The reason for that situation is that we use a dependent type to represent a type with errors. There ar

[PATCH] D70157: Align branches within 32-Byte boundary(NOP padding)

reames accepted this revision. reames added a comment. LGTM. The patch isn't perfect, but this has reached the point where landing and iterating in tree is the fastest way to convergence. To be clear, this LGTM *only* applies to the current patch contents, and the *internal only* flag names t

[PATCH] D71726: Let clang atomic builtins fetch add/sub support floating point types

jfb added subscribers: ldionne, EricWF, mclow.lists. jfb added a comment. This generally seems fine. Does it work on most backends? I want to make sure it doesn't fail in backends :) Also, @ldionne / @EricWF / @mclow.lists do you need this in libc++ for floating-point atomic support? CHANGES

[clang] b284005 - [analyzer] Add a syntactic security check for ObjC NSCoder API.

Author: Artem Dergachev Date: 2019-12-19T14:54:29-08:00 New Revision: b284005072122fe4af879725e3c8090009f89ca0 URL: https://github.com/llvm/llvm-project/commit/b284005072122fe4af879725e3c8090009f89ca0 DIFF: https://github.com/llvm/llvm-project/commit/b284005072122fe4af879725e3c8090009f89ca0.dif

[PATCH] D71475: [WIP][OPENMP] Try to fix linear clause crash by emitting alloca for step

jdoerfert added a comment. In D71475#1791825 , @cchen wrote: > [...] I found it really hard to update the tests by reading the diagnostic > message. Can I refactor the test a bit (like separate each openmp portion to > be CK1, CK2...) so that I can modif

[PATCH] D71728: [analyzer] Add a syntactic security check for ObjC NSCoder API.

This revision was automatically updated to reflect the committed changes. Closed by commit rGb28400507212: [analyzer] Add a syntactic security check for ObjC NSCoder API. (authored by dergachev.a). Changed prior to commit: https://reviews.llvm.org/D71728?vs=234786&id=234794#toc Repository: r

[clang] f4a45c2 - Fix crash on init-capture packs where the type of the initializer is non-dependent.

Author: Richard Smith Date: 2019-12-19T15:20:10-08:00 New Revision: f4a45c2ce4ce2a7a33d5773048682e65f348a486 URL: https://github.com/llvm/llvm-project/commit/f4a45c2ce4ce2a7a33d5773048682e65f348a486 DIFF: https://github.com/llvm/llvm-project/commit/f4a45c2ce4ce2a7a33d5773048682e65f348a486.diff

[PATCH] D71698: [AArch64][SVE] Add intrinsic for non-faulting loads

efriedma added inline comments. Comment at: llvm/lib/Target/AArch64/SVEInstrFormats.td:5333 + // We need a layer of indirection because early machine code passes balk at + // physical register (i.e. FFR) uses that have no previous definition. + let hasSideEffects = 1, hasNoSch

  1   2   >