[PATCH] D87981: [X86] AMX programming model.

pengfei accepted this revision. pengfei added a comment. LGTM. I think we can land this patch as a beginning. Cheers~ Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87981/new/ https://reviews.llvm.org/D87981

[PATCH] D88394: [Driver][M68k] (Patch 8/8) Add driver support for M68k

bruno accepted this revision. bruno added a comment. This revision is now accepted and ready to land. LGTM, it would be great if someone else stamps this too, in case I missed something. Comment at: clang/include/clang/Driver/Options.td:164 +def m_m68k_Features_Group:

[PATCH] D92996: [VE] Remove -faddrsig and -fnoaddrsig tests

MaskRay added inline comments. Comment at: clang/test/Driver/ve-toolchain.cpp:72 // RUN: %clangxx -### -target ve %s 2>&1 | FileCheck -check-prefix=DEFINITARRAY %s // DEFINITARRAY: clang{{.*}} "-cc1" Most newer toolchains default to -fuse-init-array. There

[PATCH] D92160: [clang] Fix wrong FDs are used for files with same name in Tooling

OikawaKirie added a comment. Replies from the original author Hao Zhang (Sorry for the wrong email address in the previous reply.) -- Thanks for your reply. I noticed this problem when I was using clang-check to analyze a project

[PATCH] D92427: [OPENMP51] Add present modifier in defaultmap clause

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

[PATCH] D87981: [X86] AMX programming model.

LuoYuanke added a comment. Hi, I'm going to land the patch if there is no objection. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87981/new/ https://reviews.llvm.org/D87981 ___ cfe-commits mailing list

[PATCH] D93002: [NPM] Support -fmerge-functions

aeubanks created this revision. aeubanks added reviewers: ychen, asbirlea, nikic. Herald added a subscriber: hiraditya. aeubanks requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. I tried to put it in the same place in the

[PATCH] D92992: [clang-tidy] Add make_unique_for_overwrite/make_shared_for_overwrite check.

Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/docs/ReleaseNotes.rst:143 +- New :doc:`modernize-make-shared-for-overwrite + ` check. Please keep alphabetical order. Comment at:

[PATCH] D92800: [Clang] Add disable-merge function attribute which generates nomerge function attribute in IR

zequanwu updated this revision to Diff 310760. zequanwu added a comment. start a new diff to make nomerge attribute a function attribute as well as a statement attribute. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92800/new/

[PATCH] D92996: [VE] Remove -faddrsig and -fnoaddrsig tests

kaz7 created this revision. kaz7 added reviewers: MaskRay, simoll, k-ishizaka. kaz7 added projects: VE, clang. kaz7 requested review of this revision. Herald added a subscriber: cfe-commits. Remove explicitly declared -faddrsig and -fnoaddrsig option tests since those are already tested in

[PATCH] D88394: [Driver][M68k] (Patch 8/8) Add driver support for M68k

myhsu updated this revision to Diff 310755. myhsu marked an inline comment as done. myhsu added a comment. - [NFC] Fix minor formatting CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88394/new/ https://reviews.llvm.org/D88394 Files: clang/include/clang/Driver/Options.td

[PATCH] D92954: [clang-offload-bundler] Add option -list

yaxunl updated this revision to Diff 310751. yaxunl added a comment. Remove unnecessary formatting changes. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92954/new/ https://reviews.llvm.org/D92954 Files: clang/test/Driver/clang-offload-bundler.c

[PATCH] D92995: Indicate that AddressSanitizer is supported on Apple Silicon.

frewsxcv created this revision. frewsxcv requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Confirmed that AddressSanitizer works on my new Macbook Air. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D92995 Files:

[PATCH] D92954: [clang-offload-bundler] Add option -list

yaxunl added inline comments. Comment at: clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp:763 + +// Create an intermediate temporary file for reading the bundles. +TempFileHandlerRAII TempFiles; tra wrote: > Having to create a temporary file in

[PATCH] D92954: [clang-offload-bundler] Add option -list

yaxunl updated this revision to Diff 310748. yaxunl marked an inline comment as done. yaxunl added a comment. Revised by Artem's comments: removing unnecessary output to temporary file, extract forEachBundle. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92954/new/

[PATCH] D91488: Consider reference, pointer, and pointer-to-membber TemplateArguments to be different if they have different types.

rjmccall added inline comments. Comment at: clang/lib/AST/ItaniumMangle.cpp:3758 TemplateArgument TA(T->getNumBitsExpr()); - mangleTemplateArgs(, 1); + mangleTemplateArgs(TemplateName(), , 1); if (T->isUnsigned()) Passing a null TemplateName here causes

[PATCH] D92992: [clang-tidy] Add make_unique_for_overwrite/make_shared_for_overwrite check.

ckennelly created this revision. ckennelly added reviewers: hokein, njames93, alexfh, aaron.ballman. Herald added subscribers: xazax.hun, mgorny. ckennelly requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. std::make_unique_for_overwrite and

[PATCH] D92160: [clang] Fix wrong FDs are used for files with same name in Tooling

OikawaKirie added a comment. In D92160#2443405 , @dexonsmith wrote: > In D92160#2443413 , @dexonsmith wrote: > Replies on my own: > Can you tell me more about the scenario this happens? You can first

[PATCH] D92990: [clangd] Provide suggestions with invalid config keys

njames93 created this revision. njames93 added reviewers: sammccall, kadircet. Herald added subscribers: usaxena95, arphaman. njames93 requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang. Update the config file warning

[PATCH] D91488: Consider reference, pointer, and pointer-to-membber TemplateArguments to be different if they have different types.

rsmith updated this revision to Diff 310741. rsmith added a comment. Rebase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91488/new/ https://reviews.llvm.org/D91488 Files: clang/include/clang/Basic/LangOptions.h

[clang] fc39425 - [NFCI] Add a missing triple in clang/test/CodeGen/ppc64le-varargs-f128.c

Author: Yuanfang Chen Date: 2020-12-09T18:17:34-08:00 New Revision: fc3942526f5c048759e90d10289ecef75410f728 URL: https://github.com/llvm/llvm-project/commit/fc3942526f5c048759e90d10289ecef75410f728 DIFF: https://github.com/llvm/llvm-project/commit/fc3942526f5c048759e90d10289ecef75410f728.diff

[clang] 7127fd1 - MSABI: Basic mangling for access to member subobjects in a class

Author: Richard Smith Date: 2020-12-09T18:08:49-08:00 New Revision: 7127fd1786e607990ada5ade2bf473e6cad68d9d URL: https://github.com/llvm/llvm-project/commit/7127fd1786e607990ada5ade2bf473e6cad68d9d DIFF: https://github.com/llvm/llvm-project/commit/7127fd1786e607990ada5ade2bf473e6cad68d9d.diff

[PATCH] D92662: [Clang][Coroutine] Drop const attribute on pthread_self when coroutine is enabled

MaskRay added a comment. In D92662#2444196 , @lxfind wrote: > In D92662#2443970 , @MaskRay wrote: > >> If the attribute is not suitable, glibc should drop it. The compiler can add >> `readnone`/`readonly` if

[PATCH] D92006: Refactoring the attrubute plugin example to fit the new API

psionic12 marked 5 inline comments as done. psionic12 added inline comments. Comment at: clang/test/Frontend/plugin-attribute.cpp:1 -// RUN: %clang -fplugin=%llvmshlibdir/Attribute%pluginext -emit-llvm -S %s -o - 2>&1 | FileCheck %s --check-prefix=ATTRIBUTE -// RUN: not %clang

[PATCH] D91442: [clang][Driver] Handle risvc in Baremetal.cpp.

MaskRay added a comment. This change has been failing on my machine for the past two weeks. `riscv32-toolchain-extra.c` and `riscv64-toolchain-extra.c` are not hermit. I have installed `/usr/lib/gcc-cross/riscv64-linux-gnu/10` (gcc-10-riscv64-linux-gnu and its dependencies) on my machine and

[PATCH] D92386: [VE] Add standard include path and library path for C++

kaz7 added a comment. Thank you for suggestions. I'll submit patches to improve it. Comment at: clang/test/Driver/ve-toolchain.cpp:13 + +// RUN: %clangxx -### -target ve %s 2>&1 | FileCheck -check-prefix=DYNLINKER %s +// DYNLINKER: nld{{.*}} "-dynamic-linker"

[PATCH] D92986: [clangd][NFC] Remove unnecessary vector.

njames93 created this revision. njames93 added reviewers: kbobyrev, sammccall. Herald added subscribers: usaxena95, kadircet, arphaman. njames93 requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang. As pointed out in

[PATCH] D78760: Check a class doesn't have a dependent type before iterating over its base classes

ahatanak updated this revision to Diff 310728. ahatanak added a comment. Rebase and ping. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78760/new/ https://reviews.llvm.org/D78760 Files: clang/lib/AST/DeclCXX.cpp clang/lib/Sema/SemaDeclCXX.cpp

[PATCH] D92386: [VE] Add standard include path and library path for C++

MaskRay added inline comments. Comment at: clang/test/Driver/ve-toolchain.cpp:13 + +// RUN: %clangxx -### -target ve %s 2>&1 | FileCheck -check-prefix=DYNLINKER %s +// DYNLINKER: nld{{.*}} "-dynamic-linker" "/opt/nec/ve/lib/ld-linux-ve.so.1" Many linker options

[clang] 880aa6a - [test] Fix test/Driver/ve-toolchain.cpp

Author: Fangrui Song Date: 2020-12-09T17:26:22-08:00 New Revision: 880aa6ac668b0c0aefb7c7b831e7127c3c54ee9b URL: https://github.com/llvm/llvm-project/commit/880aa6ac668b0c0aefb7c7b831e7127c3c54ee9b DIFF: https://github.com/llvm/llvm-project/commit/880aa6ac668b0c0aefb7c7b831e7127c3c54ee9b.diff

[PATCH] D91979: [Clang][Attr] Introduce the `assume` function attribute

jdoerfert updated this revision to Diff 310727. jdoerfert added a comment. minor spelling improvements Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91979/new/ https://reviews.llvm.org/D91979 Files: clang/docs/LanguageExtensions.rst

[PATCH] D91979: [Clang][Attr] Introduce the `assume` function attribute

jdoerfert added a comment. I added the warning and typo recognition as discussed in IRC. We list known assumption strings in the documentation and SemaDeclAttr. As we add new ones we should include a short description or link to their meaning. The ones I added so far are OpenMP specific

[PATCH] D92915: [Driver] Add -f[no-]legacy-pass-manager to supersede -f[no-]experimental-new-pass-manager

MaskRay marked an inline comment as done. MaskRay added inline comments. Comment at: clang/include/clang/Driver/Options.td:1354-1356 +defm legacy_pass_manager : BooleanMarshalledFFlag<"legacy-pass-manager", "CodeGenOpts.LegacyPassManager", +

[clang] 754d1d3 - [test] Fix Misc/time-passes.c

Author: Fangrui Song Date: 2020-12-09T17:17:28-08:00 New Revision: 754d1d3d52b7145acc9fd440e8ecc0c10d9e966d URL: https://github.com/llvm/llvm-project/commit/754d1d3d52b7145acc9fd440e8ecc0c10d9e966d DIFF: https://github.com/llvm/llvm-project/commit/754d1d3d52b7145acc9fd440e8ecc0c10d9e966d.diff

[PATCH] D92915: [Driver] Add -f[no-]legacy-pass-manager to supersede -f[no-]experimental-new-pass-manager

aeubanks added a comment. In D92915#2444287 , @thakis wrote: > Seems to break tests: http://45.33.8.238/linux/34907/step_7.txt that test fails when the NPM is turned on and lit isn't aware of it :) Comment at:

[PATCH] D91979: [Clang][Attr] Introduce the `assume` function attribute

jdoerfert updated this revision to Diff 310724. jdoerfert added a comment. List known assumptions and warn if unknown. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91979/new/ https://reviews.llvm.org/D91979 Files:

[PATCH] D92381: [clangd] Extract per-dir CDB cache to its own threadsafe class. NFC

thakis added a comment. Reverted in de4f5519015cc97f28718d90cc6dac73c0a15161 for now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92381/new/ https://reviews.llvm.org/D92381

[clang-tools-extra] de4f551 - Revert "[clangd] Extract per-dir CDB cache to its own threadsafe class. NFC"

Author: Nico Weber Date: 2020-12-09T20:11:19-05:00 New Revision: de4f5519015cc97f28718d90cc6dac73c0a15161 URL: https://github.com/llvm/llvm-project/commit/de4f5519015cc97f28718d90cc6dac73c0a15161 DIFF: https://github.com/llvm/llvm-project/commit/de4f5519015cc97f28718d90cc6dac73c0a15161.diff

[clang] 872bf68 - Default CodeGenOptions::LegacyPassManager to !LLVM_ENABLE_NEW_PASS_MANAGER

Author: Fangrui Song Date: 2020-12-09T17:10:17-08:00 New Revision: 872bf680011a71b73e9bd353449d9c42ebf6b8f6 URL: https://github.com/llvm/llvm-project/commit/872bf680011a71b73e9bd353449d9c42ebf6b8f6 DIFF: https://github.com/llvm/llvm-project/commit/872bf680011a71b73e9bd353449d9c42ebf6b8f6.diff

[PATCH] D92915: [Driver] Add -f[no-]legacy-pass-manager to supersede -f[no-]experimental-new-pass-manager

thakis added a comment. Seems to break tests: http://45.33.8.238/linux/34907/step_7.txt Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92915/new/ https://reviews.llvm.org/D92915 ___ cfe-commits mailing

[PATCH] D92984: SourceManager: Change getOrCreateFileID API to take FileEntryRef, NFC

dexonsmith created this revision. dexonsmith added reviewers: arphaman, jansvoboda11, JDevlieghere. Herald added subscribers: ributzka, jfb. dexonsmith requested review of this revision. Herald added a project: clang. Update `SourceManager::getOrCreateFileID` to take a `FileEntryRef` and migrate

[clang-tools-extra] 028e55d - clangd: Migrate to FileEntryRef in TweakTests, NFC

Author: Duncan P. N. Exon Smith Date: 2020-12-09T17:00:42-08:00 New Revision: 028e55d2d411f19e193e3d5df589162a317ab5f1 URL: https://github.com/llvm/llvm-project/commit/028e55d2d411f19e193e3d5df589162a317ab5f1 DIFF:

[PATCH] D92917: [Driver] Pass -plugin-opt=no-new-pass-manager to the linker

MaskRay abandoned this revision. MaskRay added a comment. I just merged this into D92915 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92917/new/ https://reviews.llvm.org/D92917

[PATCH] D92915: [Driver] Add -f[no-]legacy-pass-manager to supersede -f[no-]experimental-new-pass-manager

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGf9c0d1b0562c: [Driver] Add -f[no-]legacy-pass-manager to supersede -f[no-]experimental-new… (authored by MaskRay). Repository: rG LLVM Github

[clang] f9c0d1b - [Driver] Add -f[no-]legacy-pass-manager to supersede -f[no-]experimental-new-pass-manager

Author: Fangrui Song Date: 2020-12-09T16:57:36-08:00 New Revision: f9c0d1b0562c4789638a91ce97f1c2d99608005b URL: https://github.com/llvm/llvm-project/commit/f9c0d1b0562c4789638a91ce97f1c2d99608005b DIFF: https://github.com/llvm/llvm-project/commit/f9c0d1b0562c4789638a91ce97f1c2d99608005b.diff

[PATCH] D92915: [Driver] Add -f[no-]legacy-pass-manager to supersede -f[no-]experimental-new-pass-manager

MaskRay updated this revision to Diff 310720. MaskRay edited the summary of this revision. MaskRay added a comment. -plugin-opt=no-new-pass-manager => -plugin-opt=legacy-pass-manager Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92915/new/

[PATCH] D92915: [Driver] Add -f[no-]legacy-pass-manager to supersede -f[no-]experimental-new-pass-manager

aeubanks added a comment. In D92915#2444242 , @MaskRay wrote: > Thanks! > > @rsmith highlit the coroutine problem > https://bugs.llvm.org/show_bug.cgi?id=48190 (" ... it breaks support for a > C++20 language feature (coroutines)") > > Perhaps this means

[PATCH] D92915: [Driver] Add -f[no-]legacy-pass-manager to supersede -f[no-]experimental-new-pass-manager

MaskRay added a comment. Thanks! @rsmith highlit the coroutine problem https://bugs.llvm.org/show_bug.cgi?id=48190 (" ... it breaks support for a C++20 language feature (coroutines)") Perhaps this means that "EXPERIMENTAL" in `ENABLE_EXPERIMENTAL_NEW_PASS_MANAGER` should not be removed until

[PATCH] D92983: SourceManager: Migrate to FileEntryRef in getOrCreateContentCache, NFC

dexonsmith created this revision. dexonsmith added reviewers: arphaman, jansvoboda11. Herald added a subscriber: ributzka. dexonsmith requested review of this revision. Herald added a project: clang. Change `SourceManager::getOrCreateContentCache` to take a `FileEntryRef` and update call sites

[PATCH] D92617: [DWARF] Allow toolchain to adjust specified DWARF version.

This revision was automatically updated to reflect the committed changes. Closed by commit rG016e4ebfde28: [DWARF] Allow toolchain to adjust specified DWARF version. (authored by tra). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92617/new/

[clang] 016e4eb - [DWARF] Allow toolchain to adjust specified DWARF version.

Author: Artem Belevich Date: 2020-12-09T16:34:34-08:00 New Revision: 016e4ebfde28d6bb1ab6399fc8abd8cfc6a1d9fd URL: https://github.com/llvm/llvm-project/commit/016e4ebfde28d6bb1ab6399fc8abd8cfc6a1d9fd DIFF:

[clang] 2ea8c69 - Tooling: Migrate to FileEntryRef in clang::tooling::applyAllReplacements, NFC

Author: Duncan P. N. Exon Smith Date: 2020-12-09T16:31:38-08:00 New Revision: 2ea8c69eff07c491c7cb14dd46afd30529b89d91 URL: https://github.com/llvm/llvm-project/commit/2ea8c69eff07c491c7cb14dd46afd30529b89d91 DIFF:

[PATCH] D92975: Lex: Migrate HeaderSearch::LoadedModuleMaps to FileEntryRef

dexonsmith updated this revision to Diff 310713. dexonsmith added a comment. I must have just been building `clangLex` before, since I missed (until now) updating the other API users. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92975/new/ https://reviews.llvm.org/D92975 Files:

[PATCH] D92662: [Clang][Coroutine] Drop const attribute on pthread_self when coroutine is enabled

lxfind added a comment. In D92662#2443970 , @MaskRay wrote: > If the attribute is not suitable, glibc should drop it. The compiler can add > `readnone`/`readonly` if appropriate. It's a C library interface though, and Coroutine is likely too new for

[PATCH] D92808: [ObjC][ARC] Annotate calls with attributes instead of emitting retainRV or claimRV calls in the IR

ahatanak updated this revision to Diff 310709. ahatanak added a comment. Elaborate on when `createCallInstWithColors` should be called. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92808/new/ https://reviews.llvm.org/D92808 Files:

[PATCH] D90159: [DDG] Data Dependence Graph - DOT printer

Meinersbur accepted this revision. Meinersbur added a comment. This revision is now accepted and ready to land. Thanks, LGTM. Comment at: llvm/include/llvm/Analysis/DDG.h:489 + }, + [&] { OS << ", "; }); + return OS.str(); `llvm::interleaveComma`

[clang] 347e1f6 - Frontend: Use a getVirtualFileRef for a named pipe main file, NFC

Author: Duncan P. N. Exon Smith Date: 2020-12-09T16:17:14-08:00 New Revision: 347e1f621355d9b58901f3579fde91783c24ca29 URL: https://github.com/llvm/llvm-project/commit/347e1f621355d9b58901f3579fde91783c24ca29 DIFF:

[PATCH] D85788: [Clang test] Update to allow passing extra default clang arguments in use_clang

guiand added a comment. This whole thing is a little unfortunate, but maybe a better substitution would be leaving `%clang` as referring to the pure clang binary, with default arguments. Then we can have a `%clang_cc` which may only be used for a standard C compiler invocation. Returning to

[PATCH] D92808: [ObjC][ARC] Annotate calls with attributes instead of emitting retainRV or claimRV calls in the IR

ahatanak added a comment. ToT clang/llvm (without this patch) is turning the call to `@objc_unsafeClaimAutoreleasedReturnValue` in the EH cleanup block into an unreachable when compiling the following code. It looks like this happens because the inliner isn't adding the funclet token, but it's

[PATCH] D92967: Tooling: Migrate some tests to FileEntryRef, NFC

dexonsmith updated this revision to Diff 310702. dexonsmith retitled this revision from "Tooling: Migrate RewriterTestContext to FileEntryRef, NFC" to "Tooling: Migrate some tests to FileEntryRef, NFC". dexonsmith edited the summary of this revision. dexonsmith added a comment. Found another

[PATCH] D92600: [ASTImporter] Add support for importing GenericSelectionExpr AST nodes.

tmroeder updated this revision to Diff 310701. tmroeder added a comment. Moved the result-dependent tests into a different test class, one that explicitly uses templates already. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92600/new/

[PATCH] D92808: [ObjC][ARC] Annotate calls with attributes instead of emitting retainRV or claimRV calls in the IR

ahatanak updated this revision to Diff 310693. ahatanak edited the summary of this revision. ahatanak added a comment. Enable the optimization on Windows. Move `createCallInst` defined in ObjCARCContract.cpp to ObjCARC.h and use it to create calls to retainRV/claimRV with funclet tokens when

[PATCH] D92977: [clangd] Improve hover and goToDefinition on auto and dectype

qchateau created this revision. Herald added subscribers: usaxena95, kadircet, arphaman. qchateau requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang. Make the hover content on auto and decltype look like the hover

[PATCH] D92962: [ASTImporter] Fix import of a typedef that has an attribute

shafik added a comment. LGTM Comment at: clang/unittests/AST/ASTImporterTest.cpp:6098 + FirstDeclMatcher().match(TU, typedefDecl(hasName("X"))); + auto *ToD = Import(FromD, Lang_CXX17); + ASSERT_TRUE(ToD); We can't check the attribute is present?

[clang] 75a95bc - clang-format: Migrate createInMemoryFile to FileEntryRef, NFC

Author: Duncan P. N. Exon Smith Date: 2020-12-09T15:00:53-08:00 New Revision: 75a95bc80eae0c539b342f2cb19a92a0de02d2e1 URL: https://github.com/llvm/llvm-project/commit/75a95bc80eae0c539b342f2cb19a92a0de02d2e1 DIFF:

[PATCH] D92975: Lex: Migrate HeaderSearch::LoadedModuleMaps to FileEntryRef

dexonsmith created this revision. dexonsmith added reviewers: arphaman, jansvoboda11. Herald added a subscriber: ributzka. dexonsmith requested review of this revision. Herald added a project: clang. Migrate `HeaderSearch::LoadedModuleMaps` and a number of APIs over to `FileEntryRef`. This should

[PATCH] D92662: [Clang][Coroutine] Drop const attribute on pthread_self when coroutine is enabled

MaskRay added a comment. It is worth requesting clarification on GCC `__attribute__((const))`: (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66512) If the attribute is not suitable, glibc should drop it. The compiler can add `readnone`/`readonly` if appropriate. Repository: rG LLVM Github

[PATCH] D90889: Remove RemappedFiles param from ASTUnit::LoadFromASTFile, NFC

This revision was automatically updated to reflect the committed changes. Closed by commit rGc3ff9939bf7e: Remove RemappedFiles param from ASTUnit::LoadFromASTFile, NFC (authored by dexonsmith). Herald added a project: clang. Changed prior to commit:

[clang] c3ff993 - Remove RemappedFiles param from ASTUnit::LoadFromASTFile, NFC

Author: Duncan P. N. Exon Smith Date: 2020-12-09T14:44:31-08:00 New Revision: c3ff9939bf7efeb11da49f100a277b4d8bbeff9f URL: https://github.com/llvm/llvm-project/commit/c3ff9939bf7efeb11da49f100a277b4d8bbeff9f DIFF:

[clang] 898d61b - ARCMigrate: Migrate ObjCMT.cpp over to FileEntryRef

Author: Duncan P. N. Exon Smith Date: 2020-12-09T14:16:05-08:00 New Revision: 898d61b3cff5d79870d964c76d82764fef73efbb URL: https://github.com/llvm/llvm-project/commit/898d61b3cff5d79870d964c76d82764fef73efbb DIFF:

[PATCH] D92678: ARCMigrate: Migrate ObjCMT.cpp over to FileEntryRef

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG898d61b3cff5: ARCMigrate: Migrate ObjCMT.cpp over to FileEntryRef (authored by dexonsmith). Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D92721: [PATCH] [clang] Create SPIRABIInfo to enable SPIR_FUNC calling convention

mibintc updated this revision to Diff 310655. mibintc added a comment. I rebased and applied clang-format patch Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92721/new/ https://reviews.llvm.org/D92721 Files: clang/lib/CodeGen/CGBuiltin.cpp

[PATCH] D92971: clang-import-test: Clean up error output for files that cannot be found

dexonsmith added a comment. If the error output changes seem imprudent, I'm happy to skip those; the only thing I need here is the migration to the other overload of `createFileID` (the "side effect"); could just call `llvm::consumeError` and be done with it. Repository: rG LLVM Github

[PATCH] D92971: clang-import-test: Clean up error output for files that cannot be found

dexonsmith created this revision. dexonsmith added reviewers: shafik, arphaman, jansvoboda11. Herald added a subscriber: ributzka. dexonsmith requested review of this revision. Herald added a project: clang. Pass on the filesystem error string `FileManager::getFileRef` in `clang-import-test`'s

[PATCH] D92915: [Driver] Add -f[no-]legacy-pass-manager to supersede -f[no-]experimental-new-pass-manager

MaskRay updated this revision to Diff 310654. MaskRay retitled this revision from "[Driver] Add -f[no-]new-pass-manager to supersede -f[no-]experimental-new-pass-manager" to "[Driver] Add -f[no-]legacy-pass-manager to supersede -f[no-]experimental-new-pass-manager". MaskRay edited the summary of

[PATCH] D92886: [Sema] Warn about unused functions even when they're inline

aaronpuchert added a comment. In D92886#2443176 , @aaronpuchert wrote: >> Non-static inline functions in C have confusing semantics. > > I'm not terribly familiar with C, but judging from C11 6.7.4 it seems that C > is a bit more permissive, but then

[PATCH] D92969: [AST] Fix a constexpr-evaluator crash on error-dependent returnstmt.

hokein created this revision. hokein added reviewers: sammccall, rsmith. hokein requested review of this revision. Herald added a project: clang. When the evaluator encounters an error-dependent returnstmt, before this patch it returned a ESR_Returned without setting the result, the callsides

[PATCH] D92915: [Driver] Add -f[no-]new-pass-manager to supersede -f[no-]experimental-new-pass-manager

rsmith added a comment. I don't like having `new` in the name of the flag; I think that's going to age badly (and I would argue it already has done; the "new pass manager" isn't really all that new any more). Given that the new pass manager is considered stable and not experimental, can we

[PATCH] D92788: [clangd] NFC: Use SmallVector where possible

kbobyrev added inline comments. Comment at: clang-tools-extra/clangd/unittests/TestTU.cpp:190 const NamedDecl (ParsedAST , llvm::StringRef QName) { - llvm::SmallVector Components; + llvm::SmallVector Components; QName.split(Components, "::"); njames93

[PATCH] D92788: [clangd] NFC: Use SmallVector where possible

kbobyrev updated this revision to Diff 310650. kbobyrev marked an inline comment as done. kbobyrev added a comment. Revert the case flagged by @njames93. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92788/new/ https://reviews.llvm.org/D92788

[PATCH] D92968: Frontend: Migrate to FileEntryRef in TextDiagnosticTest, NFC

dexonsmith created this revision. dexonsmith added reviewers: arphaman, jansvoboda11. Herald added a subscriber: ributzka. dexonsmith requested review of this revision. Herald added a project: clang. Migrate over to the `FileEntryRef` overloads of `SourceManager::createFileID` and

[PATCH] D92920: [clang-tidy] Add a diagnostic callback to parseConfiguration

njames93 added a comment. In D92920#2443578 , @aaron.ballman wrote: > Nothing is calling the new `parseConfigurationWithDiags()`, is that > intentional? Also, is there a way to add test coverage for the change? Yes it is intentional, the idea is for

[PATCH] D92198: [clangd] Log remote index connectivity status

kbobyrev updated this revision to Diff 310642. kbobyrev marked 5 inline comments as done. kbobyrev added a comment. Resolve review comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92198/new/ https://reviews.llvm.org/D92198 Files:

[PATCH] D77598: Integral template argument suffix and cast printing

rsmith added inline comments. Comment at: clang/include/clang/AST/DeclTemplate.h:213 +else if (Idx >= TPL->size()) + return false; +else { We should return `true` in this case; we don't know what the corresponding parameter is so we don't know if

[PATCH] D92600: [ASTImporter] Add support for importing GenericSelectionExpr AST nodes.

tmroeder added a comment. @aaron.ballman It looks like those template tests fail on x64 windows; this is a problem that came up with D58292 as well. It ended up getting rolled back because template-based tests failed on windows, and I had to remove the tests

[PATCH] D92967: Tooling: Migrate RewriterTestContext to FileEntryRef, NFC

dexonsmith created this revision. dexonsmith added reviewers: jansvoboda11, arphaman. Herald added a subscriber: ributzka. dexonsmith requested review of this revision. Herald added a project: clang. Migrate to the `FileEntryRef` overload of `SourceManager::createFileID` (using

[PATCH] D92680: Frontend: Migrate to FileEntryRef in CompilerInstance::InitializeSourceManager, NFC

This revision was automatically updated to reflect the committed changes. Closed by commit rGa5c89bb02195: Frontend: Migrate to FileEntryRef in CompilerInstance::InitializeSourceManager… (authored by dexonsmith). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] a5c89bb - Frontend: Migrate to FileEntryRef in CompilerInstance::InitializeSourceManager, NFC

Author: Duncan P. N. Exon Smith Date: 2020-12-09T13:28:29-08:00 New Revision: a5c89bb02195a97aa71a406d9864098c764f URL: https://github.com/llvm/llvm-project/commit/a5c89bb02195a97aa71a406d9864098c764f DIFF:

[PATCH] D92930: [Clang] Add vcmla and rotated variants for Arm ACLE.

fhahn added inline comments. Comment at: clang/lib/CodeGen/CGBuiltin.cpp:5237 NEONMAP1(vcaddq_rot90_v, arm_neon_vcadd_rot90, Add1ArgType), + NEONMAP1(vcaddq_rot90_v, arm_neon_vcadd_rot90, Add1ArgType), NEONMAP1(vcage_v, arm_neon_vacge, 0), t.p.northover

[PATCH] D92930: [Clang] Add vcmla and rotated variants for Arm ACLE.

fhahn updated this revision to Diff 310632. fhahn added a comment. Updated to use new VCMLA_ROTS multiclass to define rotated variants. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92930/new/ https://reviews.llvm.org/D92930 Files:

[clang] 8b23b3a - [NFCI] Add missing triple to several LTO tests

Author: Yuanfang Chen Date: 2020-12-09T13:13:58-08:00 New Revision: 8b23b3ab3aea28b91e5aa48f29fe9eb1828515a3 URL: https://github.com/llvm/llvm-project/commit/8b23b3ab3aea28b91e5aa48f29fe9eb1828515a3 DIFF: https://github.com/llvm/llvm-project/commit/8b23b3ab3aea28b91e5aa48f29fe9eb1828515a3.diff

[PATCH] D92617: [DWARF] Allow toolchain to adjust specified DWARF version.

dblaikie accepted this revision. dblaikie added a comment. This revision is now accepted and ready to land. Fair enough - thanks for your patience/explanations! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92617/new/

[PATCH] D92617: [DWARF] Allow toolchain to adjust specified DWARF version.

tra added inline comments. Comment at: clang/test/Driver/cuda-unsupported-debug-options.cu:18 +// RUN: %clang -### -target x86_64-linux-gnu -c %s -gdwarf-5 -gembed-source 2>&1 | FileCheck %s --check-prefix=DWARF-CLAMP +// CHECK: debug information option

[PATCH] D92920: [clang-tidy] Add a diagnostic callback to parseConfiguration

aaron.ballman added a comment. Nothing is calling the new `parseConfigurationWithDiags()`, is that intentional? Also, is there a way to add test coverage for the change? Comment at: clang-tools-extra/clang-tidy/ClangTidyOptions.cpp:400 +

[PATCH] D92617: [DWARF] Allow toolchain to adjust specified DWARF version.

tra updated this revision to Diff 310622. tra added a comment. Herald added a reviewer: jdoerfert. Herald added a subscriber: sstefan1. Reorganized tests for unsupported debug options & dwarf version clamping. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D92962: [ASTImporter] Fix import of a typedef that has an attribute

martong created this revision. martong added a reviewer: teemperor. Herald added subscribers: gamesh411, Szelethus, dkrupp, rnkovacs. Herald added a reviewer: a.sidorin. Herald added a reviewer: shafik. martong requested review of this revision. Herald added a project: clang. Herald added a

[PATCH] D91673: [PGO] Enable preinline and cleanup when optimize for size

zequanwu updated this revision to Diff 310619. zequanwu added a comment. Herald added a subscriber: steven_wu. - fix existing failed testcase. - add new testcases for npm and lpm. - add comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D92181: [clangd] NFC: Add client-side logging for remote index requests

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG5a1bc69f8110: [clangd] NFC: Add client-side logging for remote index requests (authored by kbobyrev). Repository: rG LLVM Github Monorepo

[clang-tools-extra] 5a1bc69 - [clangd] NFC: Add client-side logging for remote index requests

Author: Kirill Bobyrev Date: 2020-12-09T21:40:37+01:00 New Revision: 5a1bc69f811059b8f62d381e3526d92fffa7d91a URL: https://github.com/llvm/llvm-project/commit/5a1bc69f811059b8f62d381e3526d92fffa7d91a DIFF:

[PATCH] D92181: [clangd] NFC: Add client-side logging for remote index requests

kbobyrev updated this revision to Diff 310616. kbobyrev added a comment. Change logging format to be consistent with another patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92181/new/ https://reviews.llvm.org/D92181 Files:

  1   2   3   >