[clang-tools-extra] 076dd0a - [clangd] Speed up an unfortunate timer-based test.

Author: Sam McCall Date: 2022-05-06T16:29:10+02:00 New Revision: 076dd0a763fd803897b0f806e0e350a27d2c087c URL: https://github.com/llvm/llvm-project/commit/076dd0a763fd803897b0f806e0e350a27d2c087c DIFF: https://github.com/llvm/llvm-project/commit/076dd0a763fd803897b0f806e0e350a27d2c087c.diff LO

[PATCH] D124637: [clangd][ObjC] Filter ObjC method completions on the remaining selector

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

[PATCH] D124486: [clangd] ExtractVariable support for C and Objective-C

dgoldman marked an inline comment as done. dgoldman added a comment. Friendly ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124486/new/ https://reviews.llvm.org/D124486 ___ cfe-commits mailing list

[PATCH] D124761: [analyzer] Replace adjacent assumeInBound calls to assumeInBoundDual

martong updated this revision to Diff 427637. martong added a comment. - Rebase to parent Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124761/new/ https://reviews.llvm.org/D124761 Files: clang/include/clang/StaticAnalyzer/Core/PathSensitive/Pro

[clang-tools-extra] 4b76ba8 - [clangd] Eliminate direct usage of isAvailable() matcher. NFC

Author: Sam McCall Date: 2022-05-06T16:35:12+02:00 New Revision: 4b76ba887c04458367c37eb3458cbd113475c27f URL: https://github.com/llvm/llvm-project/commit/4b76ba887c04458367c37eb3458cbd113475c27f DIFF: https://github.com/llvm/llvm-project/commit/4b76ba887c04458367c37eb3458cbd113475c27f.diff LO

[PATCH] D121120: [clang-tidy] New check for safe usage of `std::optional` and like types.

xazax.hun added a comment. In D121120#3496597 , @ymandel wrote: > Just realized we left this open. How about a disclaimer at the top of the doc > (.rst) file noting the potential cost? Anyone using clang-tidy should be > explicitly configuring which ch

[PATCH] D125095: [Clang][AIX] Add .ref in frontend for AIX XCOFF to support `-bcdtors:csect` linker option

tingwang created this revision. tingwang added reviewers: jsji, nemanjai, shchenz, hubert.reinterpretcast, PowerPC. tingwang added projects: LLVM, clang. Herald added a project: All. tingwang requested review of this revision. Herald added a subscriber: cfe-commits. This is the frontend part of .

[PATCH] D124613: In MSVC compatibility mode, friend function declarations behave as function declarations

thakis added a comment. Thanks for the patch! Two comments, more specific, one general: 1. This broke compilation of (at least) one TU in chromium, see https://bugs.chromium.org/p/chromium/issues/detail?id=1323014#c5 . Can you look into if that's expected? (We've worked around this for now.)

[PATCH] D125092: [OpenMP] Add basic support for properly handling static libraries

jdoerfert accepted this revision. jdoerfert added a comment. This revision is now accepted and ready to land. Typo(s) in the commit message, otherwise this seems reasonable. LG Comment at: clang/tools/clang-linker-wrapper/ClangLinkerWrapper.cpp:1153 + for (auto &File : Library

[clang] f44552a - [Frontend] Fix broken createInvocation test due to bad merge

Author: Sam McCall Date: 2022-05-06T17:06:13+02:00 New Revision: f44552ab387b9087fb815251064782f8fb60e643 URL: https://github.com/llvm/llvm-project/commit/f44552ab387b9087fb815251064782f8fb60e643 DIFF: https://github.com/llvm/llvm-project/commit/f44552ab387b9087fb815251064782f8fb60e643.diff LO

[PATCH] D125012: [clang] createInvocationFromCommandLine -> createInvocation, delete former. NFC

sammccall added a comment. Sorry, bad merge and this broke everything. Fixed in f44552ab387b9087fb815251064782f8fb60e643 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125012/new/ https://reviews.llvm.org/D125012 ___

[PATCH] D123812: [CUDA] Add wrapper code generation for registering CUDA images

jdoerfert added a comment. Are there unresolved concerns we should address in this review? Comment at: clang/tools/clang-linker-wrapper/OffloadWrapper.cpp:351 +/// required for texture / surface / managed variables. +Function *createRegisterGlobalsFunction(Module &M) { +

[clang-tools-extra] edaeab6 - Revert "[clangd] Speed up an unfortunate timer-based test."

Author: Sam McCall Date: 2022-05-06T17:19:00+02:00 New Revision: edaeab664cbd1dcbd7329efa553931efcfda5e50 URL: https://github.com/llvm/llvm-project/commit/edaeab664cbd1dcbd7329efa553931efcfda5e50 DIFF: https://github.com/llvm/llvm-project/commit/edaeab664cbd1dcbd7329efa553931efcfda5e50.diff LO

[clang] e12905b - [OpenMP] Add basic support for properly handling static libraries

Author: Joseph Huber Date: 2022-05-06T11:20:58-04:00 New Revision: e12905b4d5f95952600515901b82da0a8eefcd73 URL: https://github.com/llvm/llvm-project/commit/e12905b4d5f95952600515901b82da0a8eefcd73 DIFF: https://github.com/llvm/llvm-project/commit/e12905b4d5f95952600515901b82da0a8eefcd73.diff

[PATCH] D125092: [OpenMP] Add basic support for properly handling static libraries

This revision was automatically updated to reflect the committed changes. Closed by commit rGe12905b4d5f9: [OpenMP] Add basic support for properly handling static libraries (authored by jhuber6). Changed prior to commit: https://reviews.llvm.org/D125092?vs=427618&id=427646#toc Repository: rG

[PATCH] D124702: [MSVC] Add support for pragma function

steplong added a comment. In D124702#3496534 , @hans wrote: > It needs tests for the warnings about badly formed pragmas, and for pragmas > outside file/namespace scope. Appreciate the comments. I have a couple of questions: Is the tests for badly form

[PATCH] D123676: [clang-format] Fix WhitespaceSensitiveMacros not being honoured when macro closing parenthesis is followed by a newline.

owenpan accepted this revision. owenpan 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/D123676/new/ https://reviews.llvm.org/D123676 _

[PATCH] D123685: [clang][ASTImporter] Add isNewDecl

xazax.hun added inline comments. Comment at: clang/include/clang/AST/ASTImporterSharedState.h:43 + /// Set of the newly created declarations. + llvm::DenseSet NewDecls; + ASTImporter already has something like `ImportedFromDecls`. Is that not sufficient to che

[PATCH] D123812: [CUDA] Add wrapper code generation for registering CUDA images

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

[PATCH] D123812: [CUDA] Add wrapper code generation for registering CUDA images

jhuber6 added a comment. In D123812#3496914 , @yaxunl wrote: > LGTM. Did you forget to accept the revision? D123810 and D123471 still need to be looked at, but these are mostly non-

[PATCH] D125085: [clang-format] Correctly handle SpaceBeforeParens for builtins.

owenpan added inline comments. Comment at: clang/lib/Format/TokenAnnotator.cpp:3439-3441 +(Left.is(tok::identifier) || Left.Tok.getIdentifierInfo() || + Left.isFunctionLikeKeyword() || Left.is(tok::r_paren) || + Left.isSimpleTypeSpecifier())) -

[PATCH] D124983: [HLSL] add -fcgl option flag.

python3kgae updated this revision to Diff 427656. python3kgae added a comment. Remove HelpText for alias option. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124983/new/ https://reviews.llvm.org/D124983 Files: clang/include/clang/Driver/Options

[PATCH] D123773: [clang][analyzer][ctu] Make CTU a two phase analysis

xazax.hun added inline comments. Comment at: clang/include/clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h:116 + const bool Foreign = false; // From CTU. + I feel that we use different terms for the imported declarations. Sometimes we call them `new`, so

[PATCH] D123784: [clang][analyzer][ctu] Traverse the ctu CallEnter nodes in reverse order

xazax.hun added a comment. This approach fixes the worklist for the second phase. Would it be possible to create a wrapper that reverses the order of any worklist instead of committing to one and hardcode that? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.

[PATCH] D125103: [clangd] Speed up a slow sleeping testcase.

sammccall created this revision. sammccall added a reviewer: ilya-biryukov. Herald added subscribers: usaxena95, kadircet, arphaman, javed.absar. Herald added a project: All. sammccall requested review of this revision. Herald added subscribers: cfe-commits, MaskRay. Herald added a project: clang-t

[PATCH] D124996: [clang][preprocessor] Fix unsigned-ness of utf8 char literals

tahonermann added a comment. Thanks for your continued work on this, Tim! I think this is close. I did spot one issue and added a few other comments. Comment at: clang/lib/Lex/PPExpressions.cpp:417-418 +else if (Literal.isUTF8()) + Val.setIsUnsigned(PP.getLangOpts().C

[clang-tools-extra] dd87ace - [clang-tidy][NFC] Add createChecks method that also checks for LangaugeOptions

Author: Nathan James Date: 2022-05-06T17:30:34+01:00 New Revision: dd87aceb514d1c2b9c3e48854100df8b036d718a URL: https://github.com/llvm/llvm-project/commit/dd87aceb514d1c2b9c3e48854100df8b036d718a DIFF: https://github.com/llvm/llvm-project/commit/dd87aceb514d1c2b9c3e48854100df8b036d718a.diff

[PATCH] D124320: [clang-tidy] Add createChecks method that also checks for LangaugeOptions

This revision was automatically updated to reflect the committed changes. Closed by commit rGdd87aceb514d: [clang-tidy][NFC] Add createChecks method that also checks for LangaugeOptions (authored by njames93). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm

[PATCH] D125092: [OpenMP] Add basic support for properly handling static libraries

saiislam added a comment. > Ideally we could just put this on the linker itself, but nvlink doesn't seem > to support .a files. Yes, nvlink does not support archives. So we used a wrapper to extract cubin files from the archive and pass them to nvlink. Please see, Clang Nvlink Wrapper

[PATCH] D125050: [OpenMP] Try to Infer target triples using the offloading architecture

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

[PATCH] D125092: [OpenMP] Add basic support for properly handling static libraries

jhuber6 added a comment. In D125092#3497033 , @saiislam wrote: >> Ideally we could just put this on the linker itself, but nvlink doesn't seem >> to support .a files. > > Yes, nvlink does not support archives. So we used a wrapper to extract cubin > fil

[PATCH] D125103: [clangd] Speed up a slow sleeping testcase.

ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land. Many thanks for improving the running time, this slow test has bugged me since the day I first ran it. I'd even be happy to remove it completely. It's slow, possibly flaky, so IMO

[PATCH] D125050: [OpenMP] Try to Infer target triples using the offloading architecture

yaxunl added inline comments. Comment at: clang/lib/Driver/Driver.cpp:789 + (C.getInputArgs().hasArg(options::OPT_fopenmp_targets_EQ) || + C.getInputArgs().hasArg(options::OPT_offload_arch_EQ)); + if (IsOpenMPOffloading) { probably should be C.getInp

[PATCH] D125094: [ARM][Thumb] Command-line option to ensure AAPCS compliant Frame Records

efriedma added a comment. We probably want testcases for accessing various parts of the stack frame with FP stored in r11; for example, can we correctly access arguments passed on the stack if there's stack realignment? Comment at: llvm/lib/Target/ARM/Thumb1FrameLowering.cpp:

[PATCH] D125050: [OpenMP] Try to Infer target triples using the offloading architecture

yaxunl added a comment. could you please add a test to https://github.com/llvm/llvm-project/blob/main/clang/test/Driver/hip-offload-arch.hip to make sure when -fopenmp is specified, no openmp offloading is performed? Thanks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION h

[PATCH] D123471: [CUDA] Create offloading entries when using the new driver

tra added a comment. > Type struct __tgt_offload_entry { > > void*addr; // Pointer to the offload entry info. > // (function or global) > char*name; // Name of the function or global. > size_t size; // Size of the entry info (0 if it a function)

[PATCH] D125050: [OpenMP] Try to Infer target triples using the offloading architecture

jhuber6 added inline comments. Comment at: clang/lib/Driver/Driver.cpp:789 + (C.getInputArgs().hasArg(options::OPT_fopenmp_targets_EQ) || + C.getInputArgs().hasArg(options::OPT_offload_arch_EQ)); + if (IsOpenMPOffloading) { yaxunl wrote: > probably sh

[PATCH] D124866: [CUDA][HIP] support __noinline__ as keyword

tra added a reviewer: rsmith. tra added a subscriber: rsmith. tra added a comment. > I don't know how language extensions come about in CUDA or HIP -- is there an > appropriate standards body (or something similar) that's aware of this > extension and supports it? Summoning @rsmith for his lang

[PATCH] D123471: [CUDA] Create offloading entries when using the new driver

jhuber6 added a comment. In D123471#3497169 , @tra wrote: >> Type struct __tgt_offload_entry { >> >> void*addr; // Pointer to the offload entry info. >> // (function or global) >> char*name; // Name of the funct

[PATCH] D125109: [clangd] Rewrite TweakTesting helpers to avoid reparsing the same code. NFC

sammccall created this revision. sammccall added a reviewer: ilya-biryukov. Herald added subscribers: usaxena95, kadircet, arphaman. Herald added a project: All. sammccall requested review of this revision. Herald added subscribers: cfe-commits, MaskRay. Herald added a project: clang-tools-extra.

[PATCH] D124382: [Clang] Recognize target address space in superset calculation

tra added inline comments. Comment at: clang/include/clang/AST/Type.h:508 + static_cast(LangAS::FirstTargetAddressSpace)); +// When dealing with target AS return true if: +// * A is equal to B, or jchlanda wrote: > tra wrote: > > Is t

[PATCH] D125103: [clangd] Speed up a slow sleeping testcase.

sammccall added a comment. In D125103#3497140 , @ilya-biryukov wrote: > Many thanks for improving the running time, this slow test has bugged me > since the day I first ran it. > I'd even be happy to remove it completely. It's slow, possibly flaky, so I

[PATCH] D124866: [CUDA][HIP] support __noinline__ as keyword

yaxunl marked 4 inline comments as done. yaxunl added inline comments. Comment at: clang/include/clang/Basic/Attr.td:1778-1779 def NoInline : DeclOrStmtAttr { - let Spellings = [GCC<"noinline">, CXX11<"clang", "noinline">, + let Spellings = [Keyword<"__noinline__">, GCC<"noinl

[clang-tools-extra] c468635 - [clangd] Speed up a slow sleeping testcase.

Author: Sam McCall Date: 2022-05-06T20:12:17+02:00 New Revision: c468635b7dfcec302f7688f5dcde91913524e355 URL: https://github.com/llvm/llvm-project/commit/c468635b7dfcec302f7688f5dcde91913524e355 DIFF: https://github.com/llvm/llvm-project/commit/c468635b7dfcec302f7688f5dcde91913524e355.diff LO

[PATCH] D125103: [clangd] Speed up a slow sleeping testcase.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGc468635b7dfc: [clangd] Speed up a slow sleeping testcase. (authored by sammccall). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D124866: [CUDA][HIP] support __noinline__ as keyword

yaxunl updated this revision to Diff 427685. yaxunl marked 4 inline comments as done. yaxunl added a comment. revised by Aaron's comments CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124866/new/ https://reviews.llvm.org/D124866 Files: clang/include/clang/Basic/Attr.td clang/includ

[PATCH] D124866: [CUDA][HIP] support __noinline__ as keyword

aaron.ballman added inline comments. Comment at: clang/include/clang/Basic/Features.def:274 +// CUDA/HIP Features +FEATURE(cuda_noinline_keyword, true) + yaxunl wrote: > aaron.ballman wrote: > > Do the CUDA or HIP specs define `__noinline__` as a keyword specific

[PATCH] D124983: [HLSL] add -fcgl option flag.

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

[PATCH] D117829: [Clang] Add integer mul reduction builtin

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

[PATCH] D124726: Suggest typoed directives in preprocessor conditionals

cjdb added a comment. Thanks for working on this! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124726/new/ https://reviews.llvm.org/D124726 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https:

[PATCH] D123471: [CUDA] Create offloading entries when using the new driver

tra added a comment. In D123471#3497224 , @jhuber6 wrote: > It's a little tough, I chose that format because it's exactly the same as we > use with OpenMP with a few different flags. I wish whoever initially designed > the struct made the ``reserved`` f

[PATCH] D124966: Thread safety analysis: Handle compound assignment and ->* overloads

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM! You should probably add a release note for this change so users know about it. (Do we need to update any documentation from this?) Repository: rG LLVM Github Monorepo C

[PATCH] D123471: [CUDA] Create offloading entries when using the new driver

jhuber6 added inline comments. Comment at: clang/lib/CodeGen/CGCUDARuntime.h:58-70 + OffloadRegionKernelEntry = 0x0, +}; + +/// The kind flag of the global variable entry. +enum OffloadVarEntryKindFlag : uint32_t { + /// Mark the entry as a global variable.

[PATCH] D124658: [analyzer] Canonicalize SymIntExpr so the RHS is positive when possible

tomasz-kaminski-sonarsource updated this revision to Diff 427693. tomasz-kaminski-sonarsource added a comment. I have removed the assertions and updated the code to handle both extensions of reductions of bitwith from RHS to ResultType. Expanded test, to cover the above scenarios. CHANGES SINCE

[PATCH] D124658: [analyzer] Canonicalize SymIntExpr so the RHS is positive when possible

tomasz-kaminski-sonarsource updated this revision to Diff 427695. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124658/new/ https://reviews.llvm.org/D124658 Files: clang/lib/StaticAnalyzer/Core/SimpleSValBuilder.cpp clang/test/Analysis/additive-op-on-sym-int-expr.c Index: clang/test

[clang] 3fa5eb4 - [HLSL] add -fcgl option flag.

Author: python3kgae Date: 2022-05-06T11:42:15-07:00 New Revision: 3fa5eb4cfc065b686c03f912e4414fd00a54d04e URL: https://github.com/llvm/llvm-project/commit/3fa5eb4cfc065b686c03f912e4414fd00a54d04e DIFF: https://github.com/llvm/llvm-project/commit/3fa5eb4cfc065b686c03f912e4414fd00a54d04e.diff L

[PATCH] D124983: [HLSL] add -fcgl option flag.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG3fa5eb4cfc06: [HLSL] add -fcgl option flag. (authored by python3kgae). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://re

[PATCH] D123685: [clang][ASTImporter] Add isNewDecl

martong marked 4 inline comments as done. martong added a comment. @xazax.hun Hey Gabor, thanks a lot for your time and effort for reviewing this patch set! (And of course thank you for you too @steakhal, but you've already seen most of it.) In D123685#3458329

[PATCH] D124866: [CUDA][HIP] support __noinline__ as keyword

yaxunl marked an inline comment as done. yaxunl added inline comments. Comment at: clang/include/clang/Basic/Features.def:274 +// CUDA/HIP Features +FEATURE(cuda_noinline_keyword, true) + aaron.ballman wrote: > yaxunl wrote: > > aaron.ballman wrote: > > > Do the

[PATCH] D124866: [CUDA][HIP] support __noinline__ as keyword

yaxunl updated this revision to Diff 427700. yaxunl marked an inline comment as done. yaxunl added a comment. added release note and documentation CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124866/new/ https://reviews.llvm.org/D124866 Files: clang/docs/ReleaseNotes.rst clang/inc

[PATCH] D123471: [CUDA] Create offloading entries when using the new driver

jhuber6 updated this revision to Diff 427701. jhuber6 added a comment. Changing enum values from a bitfield to simple enumeration. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123471/new/ https://reviews.llvm.org/D123471 Files: clang/include/cl

[PATCH] D123685: [clang][ASTImporter] Add isNewDecl

xazax.hun accepted this revision. xazax.hun added a comment. This revision is now accepted and ready to land. Thanks, this looks good to me! Comment at: clang/include/clang/AST/ASTImporterSharedState.h:43 + /// Set of the newly created declarations. + llvm::DenseSet NewDecls;

[PATCH] D124658: [analyzer] Canonicalize SymIntExpr so the RHS is positive when possible

steakhal added a comment. @uabelho Could you please apply this change and check if it resolves your crashes? I'm gonna do the same on our side. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124658/new/ https://reviews.llvm.org/D124658 ___ cf

[clang] 7e63a0d - [clang-tidy] New check for safe usage of `std::optional` and like types.

Author: Yitzhak Mandelbaum Date: 2022-05-06T18:50:36Z New Revision: 7e63a0d479dd3ccce20de5cddb0f138b537c08bb URL: https://github.com/llvm/llvm-project/commit/7e63a0d479dd3ccce20de5cddb0f138b537c08bb DIFF: https://github.com/llvm/llvm-project/commit/7e63a0d479dd3ccce20de5cddb0f138b537c08bb.diff

[PATCH] D121120: [clang-tidy] New check for safe usage of `std::optional` and like types.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG7e63a0d479dd: [clang-tidy] New check for safe usage of `std::optional` and like types. (authored by ymandel). Changed prior to commit: https://rev

[PATCH] D124702: [MSVC] Add support for pragma function

steplong added inline comments. Comment at: clang/lib/Sema/SemaAttr.cpp:1079 + MSFunctionNoBuiltins.insert(MSFunctionNoBuiltins.end(), + NoBuiltins.begin(), NoBuiltins.end()); +} hans wrote: > steplong wrote: > > hans wrote: > > > Do

[PATCH] D124866: [CUDA][HIP] support __noinline__ as keyword

tra added a comment. > CUDA/HIP do not have language spec. Well. It's not completely true. CUDA programming guide does serve as the de-facto spec for CUDA. It's far from perfect, but it does mention `__noinline__` and `__forceinline__` as function qualifiers: https://docs.nvidia.com/cuda/cuda-

[PATCH] D123471: [CUDA] Create offloading entries when using the new driver

tra added inline comments. Comment at: clang/lib/CodeGen/CGCUDARuntime.h:56-70 +enum OffloadRegionEntryKindFlag : uint32_t { + /// Mark the region entry as a kernel. + OffloadRegionKernelEntry = 0x0, +}; + +/// The kind flag of the global variable entry. +

[clang-tools-extra] ec34de1 - [clang-tidy][NFC] Fix doc typo for bugprone-unchecked-optional-access

Author: Yitzhak Mandelbaum Date: 2022-05-06T19:23:43Z New Revision: ec34de1bfe5501fd3017ff867bc2481dc052788d URL: https://github.com/llvm/llvm-project/commit/ec34de1bfe5501fd3017ff867bc2481dc052788d DIFF: https://github.com/llvm/llvm-project/commit/ec34de1bfe5501fd3017ff867bc2481dc052788d.diff

[PATCH] D124790: [HLSL] Enable half type for hlsl.

python3kgae updated this revision to Diff 427711. python3kgae added a comment. Rebase for fcgl change. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124790/new/ https://reviews.llvm.org/D124790 Files: clang/include/clang/Driver/Options.td clan

[PATCH] D125120: Initial work on parameter info for function pointers

Qwinci created this revision. Herald added subscribers: usaxena95, kadircet, arphaman. Herald added a project: All. Qwinci requested review of this revision. Herald added projects: clang, clang-tools-extra. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://revi

[PATCH] D124866: [CUDA][HIP] support __noinline__ as keyword

yaxunl added a comment. In D124866#3497439 , @tra wrote: >> CUDA/HIP do not have language spec. > > Well. It's not completely true. CUDA programming guide does serve as the > de-facto spec for CUDA. It's far from perfect, but it does mention > `__noinli

[PATCH] D124658: [analyzer] Canonicalize SymIntExpr so the RHS is positive when possible

tomasz-kaminski-sonarsource updated this revision to Diff 427717. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124658/new/ https://reviews.llvm.org/D124658 Files: clang/lib/StaticAnalyzer/Core/SimpleSValBuilder.cpp clang/test/Analysis/additive-op-on-sym-int-expr.c Index: clang/test

[PATCH] D124658: [analyzer] Canonicalize SymIntExpr so the RHS is positive when possible

tomasz-kaminski-sonarsource added a comment. Added missing casts in `truncatingToUnsigned` tests. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124658/new/ https://reviews.llvm.org/D124658 ___ cfe-commits mailing list cfe-commits@lists.llvm.o

[clang] 24e9d90 - Added the brief discription about the new CSA checker.

Author: Shivam Date: 2022-05-07T01:16:22+05:30 New Revision: 24e9d90e65243fd8674bfc264c1c7d27c3cce67c URL: https://github.com/llvm/llvm-project/commit/24e9d90e65243fd8674bfc264c1c7d27c3cce67c DIFF: https://github.com/llvm/llvm-project/commit/24e9d90e65243fd8674bfc264c1c7d27c3cce67c.diff LOG: A

[PATCH] D120727: [libc++] Overhaul how we select the ABI library

ldionne updated this revision to Diff 427721. ldionne added a comment. Rebase onto main Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120727/new/ https://reviews.llvm.org/D120727 Files: clang/cmake/caches/CrossWinToARMLinux.cmake libcxx/CMakeL

[clang] b390173 - update the doc for the static analyzer checker

Author: Shivam Date: 2022-05-07T01:19:46+05:30 New Revision: b39017340806ee68b305d5c8330cbc3e1d398a4d URL: https://github.com/llvm/llvm-project/commit/b39017340806ee68b305d5c8330cbc3e1d398a4d DIFF: https://github.com/llvm/llvm-project/commit/b39017340806ee68b305d5c8330cbc3e1d398a4d.diff LOG: u

[PATCH] D124702: [MSVC] Add support for pragma function

steplong updated this revision to Diff 427716. steplong added a reviewer: aaron.ballman. steplong added a comment. Changed it to use SmallSetVector instead of SmallVector. Had to use an temporary SmallVector in AddRangeBasedNoBuiltin() because I'm not sure how to get a pointer to StringRef. getA

[clang] bbd0319 - Update ReleaseNotes.rst

Author: Shivam Date: 2022-05-07T01:20:45+05:30 New Revision: bbd031943a3d1bd72fed362ee3e8456dbb901747 URL: https://github.com/llvm/llvm-project/commit/bbd031943a3d1bd72fed362ee3e8456dbb901747 DIFF: https://github.com/llvm/llvm-project/commit/bbd031943a3d1bd72fed362ee3e8456dbb901747.diff LOG: U

[PATCH] D120489: [analyzer] Done some changes to detect Uninitialized read by the char array manipulation functions

phyBrackets added a comment. In D120489#3493313 , @martong wrote: > Maybe it is not too late to update the clang/docs/ReleaseNotes.rst? A new > checker is certainly important for the users. Many thanks! Yeah done! Thanks. Repository: rG LLVM Github

[PATCH] D123471: [CUDA] Create offloading entries when using the new driver

jhuber6 added inline comments. Comment at: clang/lib/CodeGen/CGCUDARuntime.h:56-70 +enum OffloadRegionEntryKindFlag : uint32_t { + /// Mark the region entry as a kernel. + OffloadRegionKernelEntry = 0x0, +}; + +/// The kind flag of the global variable entry.

[PATCH] D123773: [clang][analyzer][ctu] Make CTU a two phase analysis

martong marked 4 inline comments as done. martong added a comment. Thanks for the review Gabor, I'll update soon and hope I could answer your questions. Comment at: clang/include/clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h:116 + const bool Foreign = false; // From

[clang] 7f0e741 - [Driver] Pass --fix-cortex-a53-843419 automatically on Fuchsia

Author: Petr Hosek Date: 2022-05-06T13:27:10-07:00 New Revision: 7f0e741db97c64b4a566d65b878c2e0fe4dabb38 URL: https://github.com/llvm/llvm-project/commit/7f0e741db97c64b4a566d65b878c2e0fe4dabb38 DIFF: https://github.com/llvm/llvm-project/commit/7f0e741db97c64b4a566d65b878c2e0fe4dabb38.diff LO

[PATCH] D114023: [Driver] Pass --fix-cortex-a53-843419 automatically on Fuchsia

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG7f0e741db97c: [Driver] Pass --fix-cortex-a53-843419 automatically on Fuchsia (authored by phosek). Herald added a subscriber: MaskRay. Herald added a

[PATCH] D123784: [clang][analyzer][ctu] Traverse the ctu CallEnter nodes in reverse order

martong added a comment. In D123784#3496981 , @xazax.hun wrote: > This approach fixes the worklist for the second phase. Would it be possible > to create a wrapper that reverses the order of any worklist instead of > committing to one and hardcode that?

[PATCH] D123784: [clang][analyzer][ctu] Traverse the ctu CallEnter nodes in reverse order

martong marked an inline comment as done. martong added inline comments. Comment at: clang/lib/StaticAnalyzer/Core/WorkList.cpp:216-218 // Number of inserted nodes, used to emulate DFS ordering in the priority // queue when insertions are equal. + long Counter = 0; ---

[PATCH] D124719: [docs] PCH usage documentation update

tahonermann closed this revision. tahonermann added a comment. Re-closing since this review has already landed. @akyrtzi, did you see my last comment? If I don't hear back from you, I'll submit a new doc change that covers both forms of PCH use. Repository: rG LLVM Github Monorepo CHANGES S

[PATCH] D124946: [clang] serialize ORIGINAL_PCH_DIR relative to BaseDirectory

rmaz updated this revision to Diff 427731. rmaz added a comment. fix for missing temp dir in test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124946/new/ https://reviews.llvm.org/D124946 Files: clang/lib/Serialization/ASTReader.cpp clang/lib

[PATCH] D124776: [SPIR-V] Allow setting SPIR-V version via target triple

iliya-diyachkov added a comment. @Anastasia, when are you going to add support for v1.5? We would use it in the SPIR-V backend. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124776/new/ https://reviews.llvm.org/D124776 ___ cfe-commits mailin

[clang] 8477a0d - [OpenMP] Allow compiling multiple target architectures with OpenMP

Author: Joseph Huber Date: 2022-05-06T16:57:16-04:00 New Revision: 8477a0d769a0b877f14e3ec3fde576b6a3b173ce URL: https://github.com/llvm/llvm-project/commit/8477a0d769a0b877f14e3ec3fde576b6a3b173ce DIFF: https://github.com/llvm/llvm-project/commit/8477a0d769a0b877f14e3ec3fde576b6a3b173ce.diff

[clang] 509b631 - [OpenMP] Try to Infer target triples using the offloading architecture

Author: Joseph Huber Date: 2022-05-06T16:57:18-04:00 New Revision: 509b631f84e97e717a675a6ef60ae4728bb11551 URL: https://github.com/llvm/llvm-project/commit/509b631f84e97e717a675a6ef60ae4728bb11551 DIFF: https://github.com/llvm/llvm-project/commit/509b631f84e97e717a675a6ef60ae4728bb11551.diff

[PATCH] D124721: [OpenMP] Allow compiling multiple target architectures with OpenMP

This revision was automatically updated to reflect the committed changes. Closed by commit rG8477a0d769a0: [OpenMP] Allow compiling multiple target architectures with OpenMP (authored by jhuber6). Changed prior to commit: https://reviews.llvm.org/D124721?vs=426521&id=427734#toc Repository: r

[PATCH] D125050: [OpenMP] Try to Infer target triples using the offloading architecture

This revision was automatically updated to reflect the committed changes. Closed by commit rG509b631f84e9: [OpenMP] Try to Infer target triples using the offloading architecture (authored by jhuber6). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125

[PATCH] D124701: [clang] Honor __attribute__((no_builtin("foo"))) on functions

steplong updated this revision to Diff 427741. steplong added a comment. Update patch to accept __attribute__((no_builtin("*")). It adds no-builtins to the function attributes, but it still doesn't affect the calls. The calls in the function still become builtin calls. Repository: rG LLVM Gi

[clang-tools-extra] 1eb9748 - Fix check-clang-tools target after 7cc8377f2c572a919ecb

Author: Sam McCall Date: 2022-05-06T23:08:47+02:00 New Revision: 1eb97481ef8b7c8923b29d80b8b018015dd3e27c URL: https://github.com/llvm/llvm-project/commit/1eb97481ef8b7c8923b29d80b8b018015dd3e27c DIFF: https://github.com/llvm/llvm-project/commit/1eb97481ef8b7c8923b29d80b8b018015dd3e27c.diff LO

[PATCH] D124700: [AMDGPU] Add llvm.amdgcn.sched.barrier intrinsic

kerbowa updated this revision to Diff 427747. kerbowa added a comment. Herald added a subscriber: jsilvanus. Use i32. Output hex. Fix hazard rec tests for pseudo instructions. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124700/new/ https://review

[PATCH] D124966: Thread safety analysis: Handle compound assignment and ->* overloads

aaronpuchert added a comment. In D124966#3497305 , @aaron.ballman wrote: > LGTM! You should probably add a release note for this change so users know > about it. Good point, I'll add a note about compound assignment and increment/decrement. I'll leave

[PATCH] D124700: [AMDGPU] Add llvm.amdgcn.sched.barrier intrinsic

rampitec accepted this revision. rampitec added a comment. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124700/new/ https://reviews.llvm.org/D124700 ___ cfe-commits mailing list cfe-commits@lists.l

[PATCH] D124966: Thread safety analysis: Handle compound assignment and ->* overloads

aaronpuchert updated this revision to Diff 427750. aaronpuchert added a comment. Add a release note. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124966/new/ https://reviews.llvm.org/D124966 Files: clang/docs/ReleaseNotes.rst clang/lib/Analys

[PATCH] D123676: [clang-format] Fix WhitespaceSensitiveMacros not being honoured when macro closing parenthesis is followed by a newline.

ksyx accepted this revision. ksyx added a comment. Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123676/new/ https://reviews.llvm.org/D123676 ___ cfe-commits mailing list cfe-commits@lists.llvm.o

<    1   2   3   >