[PATCH] D69937: [clangd] Use name of Macro to compute its SymbolID.

hokein added a comment. In D69937#1736980 , @ilya-biryukov wrote: > Does that mean we identify each macro merely by its name? > It's not uncommon to have multiple `#define` for the same name, meaning > completely different things. > > If find

[clang] 0e70c35 - [OpenCL] Add integer builtin functions

Author: Sven van Haastregt Date: 2019-11-07T14:59:33Z New Revision: 0e70c350943f1a927f481529717c4f98a465777b URL: https://github.com/llvm/llvm-project/commit/0e70c350943f1a927f481529717c4f98a465777b DIFF: https://github.com/llvm/llvm-project/commit/0e70c350943f1a927f481529717c4f98a465777b.diff

[PATCH] D69934: [clangd] Implement rename by using SelectionTree and findExplicitReferences.

hokein updated this revision to Diff 228243. hokein marked 11 inline comments as done. hokein added a comment. simplify the testcases. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69934/new/ https://reviews.llvm.org/D69934 Files:

[PATCH] D69923: [OPENMP][DOCS] Update OpenMP status (NFC)

dreachem added a comment. I don't believe I have commit permissions on this. @ABataev, can you commit this? Thanks. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69923/new/ https://reviews.llvm.org/D69923 ___

[PATCH] D58896: Suppress -Wchar-subscripts if the index is a literal char

sberg added a comment. But how about literals like `'\x80'` where the promoted value depends on whether plain `char` is signed or unsigned? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58896/new/ https://reviews.llvm.org/D58896

[PATCH] D69952: [OPENMP50]Generalize handling of context matching/scoring.

ABataev created this revision. ABataev added a reviewer: jdoerfert. Herald added a subscriber: guansong. Herald added a project: clang. Untie context matching/scoring from the attribute for declare variant directive to simplify future uses in other context-dependent directives. Repository: rG

[PATCH] D69922: [OpenMP] Use the OpenMP-IR-Builder

jdoerfert updated this revision to Diff 228264. jdoerfert added a comment. Make the command line option shorter (mentioning openmp once should suffice) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69922/new/ https://reviews.llvm.org/D69922

[PATCH] D69785: [OpenMP] Introduce the OpenMP-IR-Builder

jdoerfert updated this revision to Diff 228259. jdoerfert added a comment. Improve type generation & handling, provide examples for function types and more attributes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69785/new/

[PATCH] D53482: Add clang-format stability check with FormatTests

MyDeveloperDay added a comment. In D53482#1737203 , @aardappel wrote: > @MyDeveloperDay thanks for your thoughts.. while `-stable` would be helpful > once you know you have the issue (or to permanently turn on in a git > pre-submit), it does not help

[PATCH] D69853: [OpenMP][NFCI] Introduce llvm/IR/OpenMPConstants.h

jdoerfert added a comment. Are there blocking issues on this one or can we go ahead and adjust minor details as we go? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69853/new/ https://reviews.llvm.org/D69853

[PATCH] D69853: [OpenMP][NFCI] Introduce llvm/IR/OpenMPConstants.h

ABataev added a comment. In D69853#1737218 , @jdoerfert wrote: > Are there blocking issues on this one or can we go ahead and adjust minor > details as we go? I still think it would be good to separate patches into the LLVM part and into the clang

[PATCH] D69923: [OPENMP][DOCS] Update OpenMP status (NFC)

This revision was automatically updated to reflect the committed changes. Closed by commit rGbcf754a3212c: [OPENMP][DOCS] Update OpenMP status (NFC) (authored by ABataev). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69923/new/

[PATCH] D69950: Reapply "Fix crash on switch conditions of non-integer types in templates"

eandrews created this revision. eandrews added reviewers: rnk, gribozavr, gribozavr2, Szelethus, erichkeane, riccibruno. This patch reapplies D61027 . When D61027 was previously committed (76945821b9cad3), buildbots failed due

[PATCH] D69922: [OpenMP] Use the OpenMP-IR-Builder

jdoerfert marked an inline comment as done. jdoerfert added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:3489 + !OMPRegionInfo->hasCancel())) { +OMPBuilder->CreateBarrier({CGF.Builder.saveIP()}, Kind, ForceSimpleCall, +

[PATCH] D69948: [Checkers] Added support for freopen to StreamChecker.

baloghadamsoftware added a comment. Thank you for working on this. Please add comments to the code and maybe also to the tests. I could not find anything in the standards about `freopen()` with null-pointer for the stream parameter. Comment at:

[PATCH] D54214: [RISCV] Set triple based on -march flag

simoncook updated this revision to Diff 228261. simoncook added a comment. Rebase. @lenary Following the discussion regarding D69383 , I think it's best for now to keep the logic just keeping `-march` directly, rather than using `getRISCVArch`. I think in the

[PATCH] D68997: Allow searching for prebuilt implicit modules.

arames updated this revision to Diff 228267. arames added a comment. Rebase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68997/new/ https://reviews.llvm.org/D68997 Files: clang/docs/Modules.rst clang/include/clang/Driver/Options.td

[PATCH] D58896: Suppress -Wchar-subscripts if the index is a literal char

edward-jones added a comment. In D58896#1737113 , @sberg wrote: > But how about literals like `'\x80'` where the promoted value depends on > whether plain `char` is signed or unsigned? If 'char' is signed and index into an array then this will

[PATCH] D69204: [OpenMP 5.0] - Extend defaultmap

cchen marked an inline comment as done. cchen added inline comments. Comment at: clang/lib/Sema/SemaOpenMP.cpp:16411-16420 +bool isDefaultmapModifier = (M == OMPC_DEFAULTMAP_MODIFIER_alloc) || +(M == OMPC_DEFAULTMAP_MODIFIER_to) || +

[clang] dd870f6 - Fix warning about unused std::unique result, erase shifted elements

Author: Reid Kleckner Date: 2019-11-07T10:39:29-08:00 New Revision: dd870f6929ee0b1dfb5fd000c7b826a1bd2d2571 URL: https://github.com/llvm/llvm-project/commit/dd870f6929ee0b1dfb5fd000c7b826a1bd2d2571 DIFF: https://github.com/llvm/llvm-project/commit/dd870f6929ee0b1dfb5fd000c7b826a1bd2d2571.diff

[PATCH] D69925: remove redundant LLVM version from version string when setting CLANG_VENDOR

arphaman added inline comments. Comment at: clang/lib/Basic/Version.cpp:143 llvm::raw_string_ostream OS(buf); #ifdef CLANG_VENDOR OS << CLANG_VENDOR; arphaman wrote: > This use of `CLANG_VENDOR` should be removed as well presumably. Please ignore my

[PATCH] D69938: [OpenCL] Use __generic addr space when generating internal representation of lambda

rjmccall added a comment. It does make logical sense to be able to qualify the call operator of a lambda. The lambda always starts as a temporary, so presumably we want the default address space qualifier on lambdas to be compatible with the address space of temporaries. However, that's

[PATCH] D69810: [OpenCL] Fix address space for base method call (PR43145)

rjmccall added inline comments. Comment at: clang/lib/CodeGen/CGCall.cpp:4091 +V = Builder.CreatePointerBitCastOrAddrSpaceCast( +V, IRFuncTy->getParamType(FirstIRArg)); + else Anastasia wrote: > svenvh wrote: > > @Anastasia

[PATCH] D69590: [RISCV] Fix ILP32D lowering for double+double/double+int return types

lenary added a comment. In D69590#1728628 , @asb wrote: > Thanks James - won't this still leave problems for structs that need > flattening? The tests in clang/test/CodeGen/riscv32-ilp32d-abi.c already include lots of cases that include flattening. I

[PATCH] D69263: [clangd] Implement cross-file rename.

ilya-biryukov added inline comments. Comment at: clang-tools-extra/clangd/ClangdServer.cpp:342 + // A snapshot of all file dirty buffers. + llvm::StringMap SnapShot = WorkScheduler.getAllFileContents(); auto Action = [File = File.str(), NewName = NewName.str(), Pos,

[PATCH] D53482: Add clang-format stability check with FormatTests

aardappel added a comment. @MyDeveloperDay thanks for your thoughts.. while `-stable` would be helpful once you know you have the issue (or to permanently turn on in a git pre-submit), it does not help developers that don't even know this is a thing, and waste a bunch of time figuring out what

[PATCH] D69204: [OpenMP 5.0] - Extend defaultmap

ABataev added inline comments. Comment at: clang/lib/Sema/SemaOpenMP.cpp:16411-16420 +bool isDefaultmapModifier = (M == OMPC_DEFAULTMAP_MODIFIER_alloc) || +(M == OMPC_DEFAULTMAP_MODIFIER_to) || +(M ==

[PATCH] D69922: [OpenMP] Use the OpenMP-IR-Builder

ABataev added inline comments. Comment at: clang/include/clang/Driver/Options.td:1643-1644 HelpText<"Emit OpenMP code only for SIMD-based constructs.">; +def fopenmp_enable_irbuilder : Flag<["-"], "fopenmp-enable-irbuilder">, Group, Flags<[CC1Option, NoArgumentUnused,

[PATCH] D68407: [WIP][RISCV] Use compiler-rt if no GCC installation detected

edward-jones updated this revision to Diff 228268. edward-jones added a comment. I've changed this to always return `ToolChain::UNW_None` from `RISCVToolChain::GetUnwindLibType` now. As a consequence we get the original behaviour of only `-lgcc` being added to the link command. CHANGES SINCE

[PATCH] D54214: [RISCV] Set triple based on -march flag

lenary accepted this revision. lenary added a comment. This revision is now accepted and ready to land. In D54214#1737255 , @simoncook wrote: > Rebase. > > @lenary Following the discussion regarding D69383 > , I think

[PATCH] D69922: [OpenMP] Use the OpenMP-IR-Builder

ABataev added inline comments. Comment at: clang/include/clang/Driver/Options.td:1643-1644 HelpText<"Emit OpenMP code only for SIMD-based constructs.">; +def fopenmp_enable_irbuilder : Flag<["-"], "fopenmp-enable-irbuilder">, Group, Flags<[CC1Option, NoArgumentUnused,

[PATCH] D69853: [OpenMP][NFCI] Introduce llvm/IR/OpenMPConstants.h

jdoerfert marked 2 inline comments as done. jdoerfert added a comment. In D69853#1737264 , @ABataev wrote: > In D69853#1737218 , @jdoerfert wrote: > > > Are there blocking issues on this one or can we go ahead and

[PATCH] D65744: [PR42707][OpenCL] Fix addr space deduction for auto

rjmccall accepted this revision. rjmccall added inline comments. This revision is now accepted and ready to land. Comment at: clang/lib/Sema/SemaDecl.cpp:6721 + if (getLangOpts().OpenCL) { + Anastasia wrote: > rjmccall wrote: > > Anastasia wrote: > > >

[clang] 90ecfa2 - Revert "[Sema] Suppress -Wchar-subscripts if the index is a literal char"

Author: Edward Jones Date: 2019-11-07T18:45:40Z New Revision: 90ecfa2f5f7feea6e3676d59fe6126e08c52d00e URL: https://github.com/llvm/llvm-project/commit/90ecfa2f5f7feea6e3676d59fe6126e08c52d00e DIFF: https://github.com/llvm/llvm-project/commit/90ecfa2f5f7feea6e3676d59fe6126e08c52d00e.diff LOG:

[PATCH] D69951: [clang-format] NFC allow Format.h to be clang-formatted but still maintain the same doc layout in ClangFormatStyleOptions.rst

MyDeveloperDay created this revision. MyDeveloperDay added reviewers: mitchell-stellar, klimek, sammccall, owenpan. MyDeveloperDay added projects: clang, clang-format. Format.h is used to generate ClangFormatStyleOptions.rst, the layout of the comments is critical to the rst file. Accidentally

[PATCH] D58896: Suppress -Wchar-subscripts if the index is a literal char

xbolva00 added a comment. Well, i am not sure if one twitter report is good motivation to criple warning. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58896/new/ https://reviews.llvm.org/D58896 ___

[PATCH] D68391: [RISCV] Improve sysroot computation if no GCC install detected

thakis added a comment. nvm, rnk just got to it in https://github.com/llvm/llvm-project/commit/f37b5c800e150ad915c4e0571edd2c92c0160d89 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68391/new/ https://reviews.llvm.org/D68391

[PATCH] D68391: [RISCV] Improve sysroot computation if no GCC install detected

thakis added a comment. Sorry, meant to also paste a link to a failing build: http://lab.llvm.org:8011/builders/clang-x64-windows-msvc/builds/11994/steps/stage%201%20check/logs/stdio Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68391/new/

[PATCH] D69952: [OPENMP50]Generalize handling of context matching/scoring.

ABataev marked 6 inline comments as done. ABataev added inline comments. Comment at: clang/include/clang/Basic/Attr.td:3310 +VariadicUnsignedArgument<"CtxSelectorSets">, +VariadicUnsignedArgument<"CtxSelectors">, VariadicStringArgument<"ImplVendors">

[PATCH] D69935: [DeclCXX] Remove unknown external linkage specifications

dblaikie added a comment. "The change introduced new external linkage languages ("C++11" and "C++14") which not supported in C++." Could you describe more what you mean by "not supported in C++"? These identifiers aren't part of the C++ standard/they're an implementation detail of the

[PATCH] D58896: Suppress -Wchar-subscripts if the index is a literal char

edward-jones added a comment. Okay I've reverted this in rG90ecfa2f5f7f . I'll make improvements and resubmit this for review. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D58896: Suppress -Wchar-subscripts if the index is a literal char

xbolva00 added a comment. Thank you! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58896/new/ https://reviews.llvm.org/D58896 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] bcf754a - [OPENMP][DOCS] Update OpenMP status (NFC)

Author: Alexey Bataev Date: 2019-11-07T11:07:56-05:00 New Revision: bcf754a3212c12ecc896bac66c599d571eca57d9 URL: https://github.com/llvm/llvm-project/commit/bcf754a3212c12ecc896bac66c599d571eca57d9 DIFF: https://github.com/llvm/llvm-project/commit/bcf754a3212c12ecc896bac66c599d571eca57d9.diff

[PATCH] D69922: [OpenMP] Use the OpenMP-IR-Builder

jdoerfert updated this revision to Diff 228270. jdoerfert added a comment. Update test attributes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69922/new/ https://reviews.llvm.org/D69922 Files: clang/include/clang/Basic/LangOptions.def

[clang] f37b5c8 - [RISCV] Fix up tests on Windows after new usage of sys::path::append

Author: Reid Kleckner Date: 2019-11-07T09:53:56-08:00 New Revision: f37b5c800e150ad915c4e0571edd2c92c0160d89 URL: https://github.com/llvm/llvm-project/commit/f37b5c800e150ad915c4e0571edd2c92c0160d89 DIFF: https://github.com/llvm/llvm-project/commit/f37b5c800e150ad915c4e0571edd2c92c0160d89.diff

[PATCH] D69822: [clang] Add new -fdebug-default-version flag.

cmtice updated this revision to Diff 228250. cmtice added a comment. Fix small typo in comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69822/new/ https://reviews.llvm.org/D69822 Files: clang/include/clang/Driver/Options.td clang/lib/Driver/ToolChains/Clang.cpp

[PATCH] D68407: [WIP][RISCV] Use compiler-rt if no GCC installation detected

edward-jones added a comment. In D68407#1732685 , @lenary wrote: > This patch is looking much better, thanks for updating it. > > Please may you clarify what RISC-V gcc does for `-lgcc`, `-lgcc_s`, > `-lgcc_eh`? Is it different to what gcc does on other

[PATCH] D69922: [OpenMP] Use the OpenMP-IR-Builder

jdoerfert marked an inline comment as done. jdoerfert added inline comments. Comment at: clang/include/clang/Driver/Options.td:1643-1644 HelpText<"Emit OpenMP code only for SIMD-based constructs.">; +def fopenmp_enable_irbuilder : Flag<["-"], "fopenmp-enable-irbuilder">,

[PATCH] D68391: [RISCV] Improve sysroot computation if no GCC install detected

thakis added a comment. The test seems to fail on Windows, could you take a look and revert if it takes a while to fix? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68391/new/ https://reviews.llvm.org/D68391

[PATCH] D69853: [OpenMP][NFCI] Introduce llvm/IR/OpenMPConstants.h

jdoerfert updated this revision to Diff 228275. jdoerfert marked an inline comment as done. jdoerfert added a comment. Use an llvm::IndexedMap instead of std::map Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69853/new/

[PATCH] D58896: Suppress -Wchar-subscripts if the index is a literal char

xbolva00 added a comment. It was said Clang does not have to match gcc 1:1. Just because someone uses weird patterns and instead of using pragma push/pop diagnostic in his/her code, should we really change it in Clang? You should provide extra flag which controls this specific behaviour (and

[PATCH] D69952: [OPENMP50]Generalize handling of context matching/scoring.

jdoerfert added inline comments. Comment at: clang/include/clang/Basic/Attr.td:3310 +VariadicUnsignedArgument<"CtxSelectorSets">, +VariadicUnsignedArgument<"CtxSelectors">, VariadicStringArgument<"ImplVendors"> Is it possible to have a

[PATCH] D68578: [HIP] Fix device stub name

rjmccall added a comment. Distinguishing between multiple symbols associated with the same source-level declaration is the purpose of the GlobalDecl abstraction. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68578/new/ https://reviews.llvm.org/D68578

[PATCH] D69925: remove redundant LLVM version from version string when setting CLANG_VENDOR

arphaman added a comment. +1, Apple's clang has an `#ifdef 0` around this code. Comment at: clang/lib/Basic/Version.cpp:143 llvm::raw_string_ostream OS(buf); #ifdef CLANG_VENDOR OS << CLANG_VENDOR; This use of `CLANG_VENDOR` should be removed as well

[clang] 118f783 - [clang-rename] Respect the traversal scope when traversing the entire AST.

Author: Haojian Wu Date: 2019-11-07T10:43:54+01:00 New Revision: 118f7836a65e864ef3c7e015d58ca370fee65e89 URL: https://github.com/llvm/llvm-project/commit/118f7836a65e864ef3c7e015d58ca370fee65e89 DIFF: https://github.com/llvm/llvm-project/commit/118f7836a65e864ef3c7e015d58ca370fee65e89.diff

[PATCH] D69813: [analyzer] CERTStrChecker: Model gets()

aaron.ballman added a comment. In D69813#1736667 , @Charusso wrote: > In D69813#1736611 , @aaron.ballman > wrote: > > > Would it make sense to use `cert.str.31.c` to remove the random dash? Would > > this also

[PATCH] D68912: Adds -Wrange-loop-analysis to -Wall

aaron.ballman accepted this revision. aaron.ballman added a comment. I spot-checked the list and definitely saw some true positives in there. I did not exhaustively check the list, however. The timing numbers look reasonable enough as well. Unless someone finds an issue with the fp rate that

[PATCH] D63960: [C++20] Add consteval-specific semantic for functions

Tyker updated this revision to Diff 228197. Tyker added a comment. minor fixes improved tests CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63960/new/ https://reviews.llvm.org/D63960 Files: clang/include/clang/AST/CXXRecordDeclDefinitionBits.def clang/include/clang/AST/DeclCXX.h

[PATCH] D68923: Don't warn about missing declarations for partial template specializations

aaron.ballman added a comment. I think this change is reasonable, but I'd like @rsmith to agree before you commit. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68923/new/ https://reviews.llvm.org/D68923

[PATCH] D69934: [clangd] Implement rename by using SelectionTree and findExplicitReferences.

ilya-biryukov added a comment. It's a bit unclear how we manage to still rename overrides. Is this handled by the USR-generating functions? Could we factor out the part that collects `NamedDecl`s and use those instead of the USRs instead? Comment at:

[PATCH] D69892: [clang-rename] Respect the traversal scope when traversing the entire AST.

This revision was automatically updated to reflect the committed changes. Closed by commit rG118f7836a65e: [clang-rename] Respect the traversal scope when traversing the entire AST. (authored by hokein). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D69813: [analyzer] CERTStrChecker: Model gets()

Charusso added a comment. In D69813#1736611 , @aaron.ballman wrote: > Would it make sense to use `cert.str.31.c` to remove the random dash? Would > this also help the user to do something like `cert.str.*.cpp`? if they want > just the CERT C++ STR

[clang-tools-extra] c5e4cf4 - [clangd] NFC, hide the internal-only utility function lex.

Author: Haojian Wu Date: 2019-11-07T10:58:09+01:00 New Revision: c5e4cf40ac459aae996180089a9831959ceb3d05 URL: https://github.com/llvm/llvm-project/commit/c5e4cf40ac459aae996180089a9831959ceb3d05 DIFF: https://github.com/llvm/llvm-project/commit/c5e4cf40ac459aae996180089a9831959ceb3d05.diff

[PATCH] D69935: [DeclCXX] Remove unknown external linkage specifications

ekatz created this revision. ekatz added reviewers: SouraVX, aprantl, dblaikie, JDevlieghere, uabelho, rsmith. ekatz added a project: clang. Herald added a subscriber: cfe-commits. Partial revert of r372681 "Support for DWARF-5 C++ language tags". The change introduced new external linkage

[PATCH] D69292: Proposal to add -Wtautological-compare to -Wall

rtrieu updated this revision to Diff 228361. rtrieu added a comment. Add -Wall tests to check that certain warning groups are active with it and a test to check all warning groups that are turned on by -Wall. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69292/new/

[PATCH] D69935: [DeclCXX] Remove unknown external linkage specifications

ekatz added a comment. Yes, these changes regard to the same thing - the removal of the "Linkage-Spec LanguageIDs". As you described it nicely, it is for linkage only, meaning C vs C++. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69935/new/

[PATCH] D69292: Proposal to add -Wtautological-compare to -Wall

xbolva00 added a comment. Thanks. Looks good. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69292/new/ https://reviews.llvm.org/D69292 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D69991: Implement __attribute__((objc_direct)), __attribute__((objc_direct_members))

MadCoder created this revision. MadCoder added reviewers: rjmccall, jfb, doug.gregor, dexonsmith. Herald added subscribers: cfe-commits, kristof.beyls. Herald added a project: clang. __attribute__((objc_direct)) is an attribute on methods declaration and implementation, categories, or extensions

[PATCH] D69990: Populate CUDA flags on FreeBSD too, as many other toolchains do.

6yearold created this revision. Herald added subscribers: cfe-commits, emaste. Herald added a project: clang. This allows `clang` to be used to compile CUDA programs. Compiled simple helloworld.cu with this. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D69990 Files:

[PATCH] D69785: [OpenMP] Introduce the OpenMP-IR-Builder

jdoerfert updated this revision to Diff 228364. jdoerfert added a comment. Do not eagerly create function declarations Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69785/new/ https://reviews.llvm.org/D69785 Files:

[PATCH] D69991: Implement __attribute__((objc_direct)), __attribute__((objc_direct_members))

MadCoder marked 2 inline comments as done. MadCoder added inline comments. Comment at: clang/lib/CodeGen/CGObjCMac.cpp:1588 // interface, we cannot perform this check. + // + // Note that for direct methods, because objc_msgSend is skipped, doh

[PATCH] D69598: Work on cleaning up denormal mode handling

spatel accepted this revision. spatel added a comment. This revision is now accepted and ready to land. LGTM - see inline for some leftover naming diffs. Comment at: clang/lib/CodeGen/CGCall.cpp:1744-1745 FuncAttrs.addAttribute("null-pointer-is-valid", "true"); -if

[PATCH] D69625: [libTooling] Support implicit coercions in Stencil's `access` combinator.

ymandel added a comment. In D69625#1730324 , @gribozavr2 wrote: > > Are you suggesting we remove text and selection entirely? > > Yes! Sounds good. I'll do that in a separate patch. I think we can abandon this one. Repository: rG LLVM Github

[PATCH] D68391: [RISCV] Improve sysroot computation if no GCC install detected

lenary accepted this revision. lenary added a comment. This revision is now accepted and ready to land. Thanks for the rebase. It looks good, let's get it committed! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68391/new/ https://reviews.llvm.org/D68391

[PATCH] D69937: [clangd] Use name of Macro to compute its SymbolID.

usaxena95 added a comment. We actually use both the name and the **source location** of the macro to calculate its ID. I see that the subject of the patch might suggest otherwise. This is a trivial change which just changes the params of the function so that users don't have to carry the

[PATCH] D69922: [OpenMP] Use the OpenMP-IR-Builder

ABataev added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:3489 + !OMPRegionInfo->hasCancel())) { +OMPBuilder->CreateBarrier({CGF.Builder.saveIP()}, Kind, ForceSimpleCall, + EmitChecks);

[PATCH] D69934: [clangd] Implement rename by using SelectionTree and findExplicitReferences.

hokein added a comment. In D69934#1736729 , @ilya-biryukov wrote: > It's a bit unclear how we manage to still rename overrides. Is this handled > by the USR-generating functions? > Could we factor out the part that collects `NamedDecl`s and use those

[clang] 0f88cae - [Analyzer] Checker for Debugging Iterator Checkers

Author: Adam Balogh Date: 2019-11-08T08:59:50+01:00 New Revision: 0f88caeef8f2d4708f442d03db7723396712a143 URL: https://github.com/llvm/llvm-project/commit/0f88caeef8f2d4708f442d03db7723396712a143 DIFF: https://github.com/llvm/llvm-project/commit/0f88caeef8f2d4708f442d03db7723396712a143.diff

[PATCH] D69938: [OpenCL] Use __generic addr space when generating internal representation of lambda

Anastasia created this revision. Anastasia added a reviewer: rjmccall. Herald added subscribers: ebevhan, yaxunl. Since lambdas are represented by callable objects, we need to adjust addr space of implicit obj parameter. This patch suggests to use `__generic` for OpenCL mode. Then any lambda

[PATCH] D69883: [OpenCL] Add math and common builtin functions

Anastasia accepted this revision. Anastasia added a comment. This revision is now accepted and ready to land. LGTM! Thanks! I guess we need to think about testing quite soon. :) Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69883/new/

[PATCH] D69937: [clangd] Use name of Macro to compute its SymbolID.

hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land. there are some unrelated changes in this patch (probably trigger by your editor setting?), though they are trivial, I would avoid these changes in a same patch. Comment at:

[clang] 6fc73f6 - [OpenCL] Add math and common builtin functions

Author: Sven van Haastregt Date: 2019-11-07T13:16:04Z New Revision: 6fc73f63660b1fbe3a1af7b3f14d6fe441e0e938 URL: https://github.com/llvm/llvm-project/commit/6fc73f63660b1fbe3a1af7b3f14d6fe441e0e938 DIFF: https://github.com/llvm/llvm-project/commit/6fc73f63660b1fbe3a1af7b3f14d6fe441e0e938.diff

[PATCH] D69908: [OpenCL] Add geometric and relational builtin functions

This revision was automatically updated to reflect the committed changes. Closed by commit rG3d30f2cff7a4: [OpenCL] Add geometric and relational builtin functions (authored by svenvh). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69908/new/

[PATCH] D69204: [OpenMP 5.0] - Extend defaultmap

ABataev added inline comments. Comment at: clang/lib/Sema/SemaOpenMP.cpp:620 + SourceLocation Loc) { +auto = getTopOfStack().DefaultmapMap[DMVC]; +DefaultmapInfo.ImplicitBehavior = DMIB; `auto` -> to real type

[PATCH] D69901: [OpenCL] Add integer functions to builtin functions

This revision was automatically updated to reflect the committed changes. Closed by commit rG0e70c350943f: [OpenCL] Add integer builtin functions (authored by svenvh). Changed prior to commit: https://reviews.llvm.org/D69901?vs=228079=228238#toc Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D69937: [clangd] Use name of Macro to compute its SymbolID.

ilya-biryukov added a comment. Sorry for not reading through and assuming the wrong thing from the title. Thanks for the explanation! LG Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69937/new/ https://reviews.llvm.org/D69937

[PATCH] D69937: [clangd] Use name of Macro to compute its SymbolID.

usaxena95 created this revision. usaxena95 added a reviewer: hokein. Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. We use the name from the IdentifierInfo of the Macro to compute its SymbolID. It is better to just take

[PATCH] D69263: [clangd] Implement cross-file rename.

hokein added inline comments. Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:764 + /*WantFormat=*/true, + [this](PathRef File) { return DraftMgr.getDraft(File); }, + [File, Params, Reply = std::move(Reply), ilya-biryukov wrote: > We

[PATCH] D69263: [clangd] Implement cross-file rename.

hokein updated this revision to Diff 228213. hokein marked 5 inline comments as done. hokein added a comment. Herald added a subscriber: javed.absar. - get dirty buffer in clangdServer layer; - save a snpatshot for all dirty buffer; Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[clang] de61aa3 - [RISCV] Improve sysroot computation if no GCC install detected

Author: Edward Jones Date: 2019-11-07T15:17:40Z New Revision: de61aa3118b9bac85c468ea7ec40604a086744f5 URL: https://github.com/llvm/llvm-project/commit/de61aa3118b9bac85c468ea7ec40604a086744f5 DIFF: https://github.com/llvm/llvm-project/commit/de61aa3118b9bac85c468ea7ec40604a086744f5.diff LOG:

[PATCH] D69901: [OpenCL] Add integer functions to builtin functions

Anastasia accepted this revision. Anastasia added a comment. This revision is now accepted and ready to land. LGTM! Thanks! Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69901/new/ https://reviews.llvm.org/D69901

[PATCH] D69922: [OpenMP] Use the OpenMP-IR-Builder

ABataev added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:3489 + !OMPRegionInfo->hasCancel())) { +OMPBuilder->CreateBarrier({CGF.Builder.saveIP()}, Kind, ForceSimpleCall, + EmitChecks);

[PATCH] D69937: [clangd] Use name of Macro to compute its SymbolID.

ilya-biryukov added a comment. Does that mean we identify each macro merely by its name? It's not uncommon to have multiple `#define` for the same name, meaning completely different things. If find references shows all of those, it's not very useful... Repository: rG LLVM Github Monorepo

[PATCH] D69922: [OpenMP] Use the OpenMP-IR-Builder

jdoerfert marked 2 inline comments as done. jdoerfert added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:3489 + !OMPRegionInfo->hasCancel())) { +OMPBuilder->CreateBarrier({CGF.Builder.saveIP()}, Kind, ForceSimpleCall, +

[PATCH] D60455: [SYCL] Implement SYCL device code outlining

bader added a comment. Hi @aaron.ballman, Thanks a lot for the comments and sorry for the long delay. We've been working on complete implementation of the SYCL 1.2.1 specification. Now I have more time to work on contributing the implementation to LLVM project. I re-based the patch and started

[clang] 7adab77 - [Sema] Suppress -Wchar-subscripts if the index is a literal char

Author: Edward Jones Date: 2019-11-07T15:45:44Z New Revision: 7adab7719e55e1b29bfd521dcc73f202139e8f41 URL: https://github.com/llvm/llvm-project/commit/7adab7719e55e1b29bfd521dcc73f202139e8f41 DIFF: https://github.com/llvm/llvm-project/commit/7adab7719e55e1b29bfd521dcc73f202139e8f41.diff LOG:

[PATCH] D58896: Suppress -Wchar-subscripts if the index is a literal char

This revision was automatically updated to reflect the committed changes. Closed by commit rG7adab7719e55: [Sema] Suppress -Wchar-subscripts if the index is a literal char (authored by edward-jones). Herald added a subscriber: simoncook. Changed prior to commit:

[PATCH] D69810: [OpenCL] Fix address space for base method call (PR43145)

Anastasia added inline comments. Comment at: clang/lib/CodeGen/CGCall.cpp:4091 +V = Builder.CreatePointerBitCastOrAddrSpaceCast( +V, IRFuncTy->getParamType(FirstIRArg)); + else svenvh wrote: > @Anastasia pointed out that the

[clang] 10e0d64 - CodeGen: set correct result for atomic compound expressions

Author: Tim Northover Date: 2019-11-07T13:36:44Z New Revision: 10e0d64337d64ebdb658bf9108bd9bb48fb5390c URL: https://github.com/llvm/llvm-project/commit/10e0d64337d64ebdb658bf9108bd9bb48fb5390c DIFF: https://github.com/llvm/llvm-project/commit/10e0d64337d64ebdb658bf9108bd9bb48fb5390c.diff

[PATCH] D67436: CodeGen: set correct result for atomic compound expressions

t.p.northover closed this revision. t.p.northover added a comment. Thanks JF. To github.com:llvm/llvm-project.git 0ec6a4882ee..10e0d64337d master -> master > Separately, does this do floating-point add / sub properly? We added them too > C++20. It looks like that already works because it

  1   2   3   >