[PATCH] D70111: [DWARF5]Addition of alignment field in the typedef for dwarf5

djtodoro added a comment. @awpandey Thanks for the patch! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70111/new/ https://reviews.llvm.org/D70111 ___ cfe-commits mailing list cfe-commits@lists.llvm.or

[PATCH] D70111: [DWARF5]Addition of alignment field in the typedef for dwarf5

djtodoro added inline comments. Comment at: llvm/lib/CodeGen/AsmPrinter/DwarfUnit.cpp:803 + if (Tag == dwarf::DW_TAG_typedef && DD->getDwarfVersion() >= 5) { +uint32_t AlignInBytes = DTy->getAlignInBytes(); Please add a comment here. Com

[PATCH] D70111: [DWARF5]Addition of alignment field in the typedef for dwarf5

awpandey created this revision. awpandey added reviewers: aprantl, dblaikie, jini.susan.george, SouraVX, alok. awpandey added a project: debug-info. Herald added subscribers: llvm-commits, cfe-commits, ormris, hiraditya. Herald added projects: clang, LLVM. This patch, adds support for DW_AT_alignm

[PATCH] D70110: [Driver][FreeBSD] Enable unwind tables on !amd64

cem created this revision. cem added reviewers: dim, emaste. Herald added subscribers: cfe-commits, krytarowski. Herald added a project: clang. There doesn't seem to be much sense in defaulting "on" unwind tables on amd64 and not on other arches. It causes surprising differences between platforms

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

rjmccall added a comment. In D69938#1741080 , @Anastasia wrote: > In D69938#1737196 , @rjmccall wrote: > > > It does make logical sense to be able to qualify the call operator of a > > lambda. The lambda always st

[PATCH] D68362: [libunwind][RISCV] Add 64-bit RISC-V support

mhorne marked 3 inline comments as done. mhorne added inline comments. Comment at: libunwind/include/__libunwind_config.h:26 #define _LIBUNWIND_HIGHEST_DWARF_REGISTER_SPARC 31 +#define _LIBUNWIND_HIGHEST_DWARF_REGISTER_RISCV 63 luismarques wrote: > The

[PATCH] D68362: [libunwind][RISCV] Add 64-bit RISC-V support

mhorne updated this revision to Diff 228799. mhorne marked an inline comment as done. mhorne added a comment. Convert checks for `__riscv_float_abi_double` to `__riscv_flen == 64`. Fix `_LIBUNWIND_HIGHEST_DWARF_REGISTER_RISCV`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68362/new/ h

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

jdoerfert updated this revision to Diff 228801. jdoerfert added a comment. Use IRBuilder for cancel barriers 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.de

[PATCH] D10833: Retrieve BinaryOperator::getOpcode and BinaryOperator::getOpcodeStr via libclang and its python interface

tannergooding added a comment. Would it be beneficial for this to be just called `getOpcode`? If it was, it could also be applied to `UnaryOperator::getOpcode`. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D10833/new/ https://reviews.llvm.org/D10833 _

[PATCH] D68682: Clang-tidy fix removals removing all non-blank text from a line should remove the line

poelmanc marked 4 inline comments as done. poelmanc added inline comments. Comment at: clang/lib/AST/CommentLexer.cpp:20 + +// Consider moving this useful function to a more general utility location. +const char *skipNewline(const char *BufferPtr, const char *BufferEnd) { ---

[PATCH] D68682: Clang-tidy fix removals removing all non-blank text from a line should remove the line

poelmanc updated this revision to Diff 228795. poelmanc edited the summary of this revision. poelmanc added a comment. Move isWhitespace and skipNewlines to clang/Basic/CharInfo.h (renaming to isWhitespaceStringRef and skipNewlinesChars to resolve name clashes) and add double-quotes around "/n"

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

jdoerfert updated this revision to Diff 228792. jdoerfert added a comment. Add cancel_barrier functionality + test, move everything to "Frontend" Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69785/new/ https://reviews.llvm.org/D69785 Files: llv

[PATCH] D70063: clang/Modules: Error if ReadASTBlock does not find the main module

dexonsmith marked 2 inline comments as done. dexonsmith added inline comments. Comment at: clang/include/clang/Basic/DiagnosticSerializationKinds.td:78 +def err_module_file_missing_definition : Error< + "module file '%0' is missing the main module's definition">, DefaultFatal;

[PATCH] D70047: [Analyzer] Use a reference in a range-based for

NoQ added a comment. Yay thanks, i always forget those :) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70047/new/ https://reviews.llvm.org/D70047 ___ cfe-commits mailing list cfe-commits@lists.llvm.or

[PATCH] D70094: [clang-tidy] new altera ID dependent backward branch check

Eugene.Zelenko added inline comments. Comment at: clang-tidy/altera/IdDependentBackwardBranchCheck.cpp:106 +if (auto *ChildExpression = dyn_cast(*i)) { + auto Result = hasIdDepVar(ChildExpression); + if (Result) { Please don't use auto when type is

[PATCH] D70101: [X86] Fix the implementation of __readcr3/__writecr3 to work in 64-bit mode

craig.topper added a comment. So in order to match the long and int behavior, I need two different macros to replace the argument? I think __INTPTR_TYPE__ will return int on 32-bit targets so I can't use that for the long case. But maybe for the __int64? CHANGES SINCE LAST ACTION https://rev

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

jdoerfert updated this revision to Diff 228788. jdoerfert marked 2 inline comments as done. jdoerfert added a comment. Create llvm/libFrontend, move code there, make shared library builds work. llvm/IR/ is an unfortunate location as we cannot use llvm utility functions without introducing depende

[PATCH] D70045: [AST] Use an explicit copy in a range-based for

aaron.ballman accepted this revision. aaron.ballman 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/D70045/new/ https://reviews.llvm.org/D70045

[PATCH] D70101: [X86] Fix the implementation of __readcr3/__writecr3 to work in 64-bit mode

craig.topper added a comment. In D70101#1741452 , @rnk wrote: > What do you think of using `__INTPTR_TYPE__` for this instead? If that > doesn't work, we could define and undefine our own macro to use as a typedef. > The resulting code should be clean en

[PATCH] D70047: [Analyzer] Use a reference in a range-based for

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM but the `&` should bind to the variable, not to the type, so that should be corrected before committing. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

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

Meinersbur added inline comments. Comment at: llvm/include/llvm/IR/OpenMPKinds.def:186 +///{ + +#ifndef OMP_IDENT_FLAG JonChesterfield wrote: > jdoerfert wrote: > > Meinersbur wrote: > > > jdoerfert wrote: > > > > JonChesterfield wrote: > > > > > Sharing constant

[PATCH] D70063: clang/Modules: Error if ReadASTBlock does not find the main module

bruno added inline comments. Comment at: clang/include/clang/Basic/DiagnosticSerializationKinds.td:78 +def err_module_file_missing_definition : Error< + "module file '%0' is missing the main module's definition">, DefaultFatal; aprantl wrote: > Should this be

[PATCH] D69959: [C-index] Fix test when using Debug target & MSVC STL

dexonsmith added a comment. In D69959#1741456 , @rnk wrote: > What do you think of making `FrontendOptions::Inputs` be a `SmallVector<*, > 0>`? It's janky and inconsistent with the rest of that struct, but it seems > less invasive in the end. +1, we us

[PATCH] D70055: clang/Modules: Clean up modules on error in ReadAST

dexonsmith closed this revision. dexonsmith added a comment. Committed c46b3a2abd38d6fecd389c97dfa7df54af77fdb9 . CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70055/new/ https://reviews.llvm.org/D70055 ___

[clang] c46b3a2 - clang/Modules: Clean up modules on error in ReadAST

Author: Duncan P. N. Exon Smith Date: 2019-11-11T16:36:02-08:00 New Revision: c46b3a2abd38d6fecd389c97dfa7df54af77fdb9 URL: https://github.com/llvm/llvm-project/commit/c46b3a2abd38d6fecd389c97dfa7df54af77fdb9 DIFF: https://github.com/llvm/llvm-project/commit/c46b3a2abd38d6fecd389c97dfa7df54af77

[PATCH] D69962: [CFG] Fix a flaky crash in CFGBlock::getLastCondition().

NoQ added a comment. In D69962#1739440 , @NoQ wrote: > In D69962#1738618 , @Szelethus wrote: > > > Nice catch! Though, wouldn't the memory sanitizer buildbots break on this > > reliably? > > > I kinda hope so; i'll

[PATCH] D69766: [Clang][MSVC] Use GetLinkerPath like the other toolchains for consistency

rnk added a comment. In D69766#1739060 , @Ericson2314 wrote: > Reading https://llvm.org/docs/DeveloperPolicy.html it looks like I need to > submit an patch email? Is that true, or can/will someone with perms see the > approval and eventually merge it fro

[PATCH] D70057: clang/Modules: Add missing diagnostics for malformed AST files

dexonsmith closed this revision. dexonsmith added a comment. Committed 8e2c192e2af8c760152ba3b28e774dbb1548e4aa . CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70057/new/ https://reviews.llvm.org/D70057 ___

[PATCH] D69869: [clang-tools-extra] fix the check for if '-latomic' is necessary

gokturk added a comment. Let me try my best to explain what's happening. The goal of this check is to determine whether passing `-latomic` is required. Let's start with the code used by `check_working_cxx_atomics64` (https://github.com/llvm/llvm-project/blob/master/llvm/cmake/modules/CheckAtomi

[PATCH] D70056: clang/Modules: Split loop in ReadAST between failable and not

dexonsmith closed this revision. dexonsmith added a comment. Committed in bfd58fc60ff4b0c081b5b489119c3798d3e2b53c and 01782c3e4df1830d7991e9edfee9119ed41e4c27

[clang] 8e2c192 - clang/Modules: Add missing diagnostics for malformed AST files

Author: Duncan P. N. Exon Smith Date: 2019-11-11T16:00:47-08:00 New Revision: 8e2c192e2af8c760152ba3b28e774dbb1548e4aa URL: https://github.com/llvm/llvm-project/commit/8e2c192e2af8c760152ba3b28e774dbb1548e4aa DIFF: https://github.com/llvm/llvm-project/commit/8e2c192e2af8c760152ba3b28e774dbb1548

[clang] bfd58fc - clang/Modules: Use range-based for in ASTReader::ReadAST, NFC

Author: Duncan P. N. Exon Smith Date: 2019-11-11T15:53:48-08:00 New Revision: bfd58fc60ff4b0c081b5b489119c3798d3e2b53c URL: https://github.com/llvm/llvm-project/commit/bfd58fc60ff4b0c081b5b489119c3798d3e2b53c DIFF: https://github.com/llvm/llvm-project/commit/bfd58fc60ff4b0c081b5b489119c3798d3e2

[clang] 01782c3 - clang/Modules: Split loop in ReadAST between failable and not

Author: Duncan P. N. Exon Smith Date: 2019-11-11T15:53:48-08:00 New Revision: 01782c3e4df1830d7991e9edfee9119ed41e4c27 URL: https://github.com/llvm/llvm-project/commit/01782c3e4df1830d7991e9edfee9119ed41e4c27 DIFF: https://github.com/llvm/llvm-project/commit/01782c3e4df1830d7991e9edfee9119ed41e

[PATCH] D70058: clang/Modules: Delay err_module_file_conflict if a diagnostic is in flight

dexonsmith closed this revision. dexonsmith added a comment. Committed as eef69021607950487a9e4110851a05abb54d0fb6 . CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70058/new/ https://reviews.llvm.org/D70058

[clang] eef6902 - clang/Modules: Delay err_module_file_conflict if a diagnostic is in flight

Author: Duncan P. N. Exon Smith Date: 2019-11-11T15:34:52-08:00 New Revision: eef69021607950487a9e4110851a05abb54d0fb6 URL: https://github.com/llvm/llvm-project/commit/eef69021607950487a9e4110851a05abb54d0fb6 DIFF: https://github.com/llvm/llvm-project/commit/eef69021607950487a9e4110851a05abb54d

[PATCH] D69959: [C-index] Fix test when using Debug target & MSVC STL

rnk added a comment. I see. You know, these things would be nicely handled if we just had a reliable asynch SEH implementation. >_> What do you think of making `FrontendOptions::Inputs` be a `SmallVector<*, 0>`? It's janky and inconsistent with the rest of that struct, but it seems less invasi

[PATCH] D70101: [X86] Fix the implementation of __readcr3/__writecr3 to work in 64-bit mode

rnk added a comment. What do you think of using `__INTPTR_TYPE__` for this instead? If that doesn't work, we could define and undefine our own macro to use as a typedef. The resulting code should be clean enough that we can generalize it to `__readcr0` etc. It looks like `long` is the correct

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

void updated this revision to Diff 228777. void added a comment. Add blurb about asm goto with output constraints to the "language extensions" documentation. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69876/new/ https://reviews.llvm.org/D69876

Re: r373159 - For P0784R7: compute whether a variable has constant destruction if it

Here's a reduced test case. ``` // OPTIONS: -x objective-c -std=gnu99 -fobjc-arc -emit-llvm @interface Foo @end Foo *foo() { static Foo *f = ((void*)0; return f; } // CHECK-NOT: cxa_guard ``` AFAIK clang should not emit C++ guard variables in non-C++ code. Is that correct? On Mon, 11 Nov 20

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

void added a comment. In D69876#1741381 , @nickdesaulniers wrote: > > Nice catch! I updated the patch with a fix. PTAL. > > Please add a test for that! I did, but the "arc" program pushed the wrong change. :-( Fixed. When are we going to use GitHub's P

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

void updated this revision to Diff 228775. void added a comment. the real update this time. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69876/new/ https://reviews.llvm.org/D69876 Files: clang/include/clang/AST/Stmt.h clang/lib/AST/Stmt.cpp

LLVM buildmaster will be updated and restarted tonight

Hello everyone, LLVM buildmaster will be updated and restarted after 6PM Pacific time today. Thanks Galina ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

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

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

jdoerfert updated this revision to Diff 228772. jdoerfert added a comment. update test case 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/include

[PATCH] D70056: clang/Modules: Split loop in ReadAST between failable and not

dexonsmith marked an inline comment as done. dexonsmith added inline comments. Comment at: clang/lib/Serialization/ASTReader.cpp:4236 + MEnd = Loaded.end(); + M != MEnd; ++M) { +ModuleFile &F = *M->Mod; apran

[PATCH] D70063: clang/Modules: Error if ReadASTBlock does not find the main module

aprantl added a comment. Do we want to bother with a testcase for this? Comment at: clang/include/clang/Basic/DiagnosticSerializationKinds.td:78 +def err_module_file_missing_definition : Error< + "module file '%0' is missing the main module's definition">, DefaultFatal;

[PATCH] D70057: clang/Modules: Add missing diagnostics for malformed AST files

aprantl accepted this revision. aprantl added inline comments. This revision is now accepted and ready to land. Comment at: clang/lib/Serialization/ASTReader.cpp:3412 + if (ParseLineTable(F, Record)) { +Error("malformed SOURCE_MANAGER_LINE_TABLE in AST file");

[PATCH] D70056: clang/Modules: Split loop in ReadAST between failable and not

aprantl accepted this revision. aprantl added inline comments. This revision is now accepted and ready to land. Comment at: clang/lib/Serialization/ASTReader.cpp:4211 + // hit errors parsing the ASTs at this point. for (SmallVectorImpl::iterator M = Loaded.begin(),

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

nickdesaulniers added a comment. In D69876#1741294 , @void wrote: > In D69876#1740286 , @jyknight wrote: > > > I think -Wuninitialized (UninitializedValues.cpp) should be taught how to > > detect the use of output

[PATCH] D67787: Add 8548 CPU definition and attributes

jhibbits added a comment. @vit9696 mind reviewing again with the added tests? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67787/new/ https://reviews.llvm.org/D67787 ___ cfe-commits mailing list cfe-c

Re: r373159 - For P0784R7: compute whether a variable has constant destruction if it

Hi Richard, I have a question about this commit. It looks like it started emitting `cxa_guard_acquire/release` calls in Objective-C code, for globals with constant initializers, causing link failures for things that don't link with libc++abi. Was that intentional? I'm still working on a reduced te

[PATCH] D69548: Give clang-tidy readability-redundant-string-init a customizable list of string types to fix

poelmanc updated this revision to Diff 228769. poelmanc added a comment. Make requested fixes to documentation. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69548/new/ https://reviews.llvm.org/D69548 Files: clang-tools-extra/clang-tidy/readabili

[PATCH] D67460: clang-tidy: modernize-use-using work with multi-argument templates

poelmanc marked 2 inline comments as done. poelmanc added a comment. Done, thanks. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67460/new/ https://reviews.llvm.org/D67460 ___ cfe-commits mailing list c

[PATCH] D69238: Fix clang-tidy readability-redundant-string-init for c++17/c++2a

poelmanc added a comment. In D69238#1740914 , @gribozavr2 wrote: > Thanks! Do you have commit access or do you need me to commit for you? I don't have commit access, if you could commit it for me that would be great. Thanks! Repository: rCTE Clang T

[PATCH] D67460: clang-tidy: modernize-use-using work with multi-argument templates

poelmanc updated this revision to Diff 228767. poelmanc added a comment. Changed post-increment/decrement to pre-increment/decrement. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67460/new/ https://reviews.llvm.org/D67460 Files: clang-tools-extr

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

MaskRay added inline comments. Comment at: clang/lib/Driver/ToolChains/RISCVToolchain.cpp:114 const char *LinkingOutput) const { - const ToolChain &ToolChain = getToolChain(); + const auto &ToolChain = getToolChain(); const Driver &D = ToolC

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

MaskRay added inline comments. Comment at: clang/lib/Driver/ToolChains/RISCVToolchain.cpp:49 + else +return ToolChain::RLT_CompilerRT; +} Ternary operator? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68407/new/ https://reviews.llvm.org/D68407

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

jdoerfert marked 2 inline comments as done. jdoerfert added inline comments. Comment at: llvm/include/llvm/IR/OpenMPConstants.h:57 +#include "llvm/IR/OpenMPKinds.def" + LLVM_MARK_AS_BITMASK_ENUM(0x7FFF) +}; ABataev wrote: > Why do you use `0x7FFF` as the

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

ABataev added inline comments. Comment at: clang/lib/Sema/SemaOpenMP.cpp:16393-16416 +static DefaultMapImplicitBehavior +getImplicitBehaviorFromModifier(OpenMPDefaultmapClauseModifier M) { + switch (M) { + case OMPC_DEFAULTMAP_MODIFIER_alloc: +return DMIB_alloc; + case OMP

[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:16393-16416 +static DefaultMapImplicitBehavior +getImplicitBehaviorFromModifier(OpenMPDefaultmapClauseModifier M) { + switch (M) { + case OMPC_DEFAULTMAP_MODIFIER_allo

[PATCH] D70099: [libcxx][test] Only call valarray::min and ::max on empty containers when testing libc++

CaseyCarter created this revision. CaseyCarter added reviewers: mclow.lists, EricWF, ldionne. Herald added a subscriber: dexonsmith. `valarray:min` and `valarray::max` have preconditions that require the `valarray` to be non-empty, so these calls are inherently non-portable. https://reviews.llv

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

ABataev added inline comments. Comment at: llvm/include/llvm/IR/OpenMPConstants.h:57 +#include "llvm/IR/OpenMPKinds.def" + LLVM_MARK_AS_BITMASK_ENUM(0x7FFF) +}; Why do you use `0x7FFF` as the largest value? Comment at: llvm/lib/IR/Open

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

void added a comment. In D69876#1740289 , @jyknight wrote: > Also, since this means we are no longer simply implementing according to > GCC's documentation, I think this means we'll need a brand new section in the > Clang docs for its inline-asm support.

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

void added a comment. In D69876#1740286 , @jyknight wrote: > I think -Wuninitialized (UninitializedValues.cpp) should be taught how to > detect the use of output variables in error blocks, at least for trivial > cases. > > Actually, for some reason -- it

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

void updated this revision to Diff 228762. void added a comment. Herald added subscribers: llvm-commits, hiraditya. Herald added a project: LLVM. Analyze "asm goto" for initialized variables. An "asm goto" statement is a terminator and thus doesn't indicate variable assignments like normal inline

[PATCH] D70052: [clang-tidy] Add misc-mutating-copy check

Eugene.Zelenko added a comment. In D70052#1741246 , @gbencze wrote: > In D70052#1740235 , @Eugene.Zelenko > wrote: > > > If this is CERT rule, why check is not placed in relevant module? > > > To be honest I was ho

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

eandrews added a comment. The patch changes type dependency of field 'std::string s' and sets it based on field type (i.e. not type-dependent). I do not know how clang-tidy works internally, but I assume checker used to skip the field earlier since it was set as dependent (based on containing

[clang] 152e83f - clang-format: fix a typo introduced by the previous change

Author: Sylvestre Ledru Date: 2019-11-11T21:52:08+01:00 New Revision: 152e83fc59af7d255df10c0f56c8fbb14dc1dea2 URL: https://github.com/llvm/llvm-project/commit/152e83fc59af7d255df10c0f56c8fbb14dc1dea2 DIFF: https://github.com/llvm/llvm-project/commit/152e83fc59af7d255df10c0f56c8fbb14dc1dea2.dif

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

dblaikie added a comment. In D69935#1739235 , @dblaikie wrote: > In D69935#1738273 , @uabelho wrote: > > > Don't you need to also remove > > > > case LinkageSpecDecl::lang_cxx_11: > > case LinkageSpecDecl::lang_

[PATCH] D70052: [clang-tidy] Add misc-mutating-copy check

gbencze updated this revision to Diff 228758. gbencze added a comment. Update documentation and release notes. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70052/new/ https://reviews.llvm.org/D70052 Files: clang-tools-extra/clang-tidy/misc/CMakeLists.txt clang-tools-extra/clang-ti

[PATCH] D70052: [clang-tidy] Add misc-mutating-copy check

gbencze added a comment. In D70052#1740235 , @Eugene.Zelenko wrote: > If this is CERT rule, why check is not placed in relevant module? To be honest I was hoping for some feedback on this as I wasn't sure what the best place for this check would be. Qu

[PATCH] D69770: Add recoverable string parsing errors to APFloat

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

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

ekatz updated this revision to Diff 228755. ekatz added a comment. Removed DWARF reference from LinkageSpecDecl::LanguageIDs. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69935/new/ https://reviews.llvm.org/D69935 Files: clang/include/clang/AST/DeclCXX.h clang/lib/AST/DeclPrinter.

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

jdoerfert updated this revision to Diff 228754. jdoerfert marked an inline comment as done. jdoerfert added a comment. Make attributes opt-in Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69785/new/ https://reviews.llvm.org/D69785 Files: llvm/in

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

This revision was automatically updated to reflect the committed changes. Closed by commit rGfde11e9f23a3: [OPENMP50]Generalize handling of context matching/scoring. (authored by ABataev). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69952/new/ htt

[clang] fde11e9 - [OPENMP50]Generalize handling of context matching/scoring.

Author: Alexey Bataev Date: 2019-11-11T14:41:10-05:00 New Revision: fde11e9f23a3bf6c78ec0bcfa92e9759ee8b5054 URL: https://github.com/llvm/llvm-project/commit/fde11e9f23a3bf6c78ec0bcfa92e9759ee8b5054 DIFF: https://github.com/llvm/llvm-project/commit/fde11e9f23a3bf6c78ec0bcfa92e9759ee8b5054.diff

[clang] f8c12ed - [OPENMP50]Add support for nested atomic and simd constructs in

Author: Alexey Bataev Date: 2019-11-11T14:28:28-05:00 New Revision: f8c12edd1a5200a2c8da754d6a3bfa7545a0 URL: https://github.com/llvm/llvm-project/commit/f8c12edd1a5200a2c8da754d6a3bfa7545a0 DIFF: https://github.com/llvm/llvm-project/commit/f8c12edd1a5200a2c8da754d6a3bfa7545a0.diff

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

jdoerfert accepted this revision. jdoerfert 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/D69952/new/ https://reviews.llvm.org/D69952

[PATCH] D70094: [clang-tidy] new altera ID dependent backward branch check

ffrankies updated this revision to Diff 228743. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70094/new/ https://reviews.llvm.org/D70094 Files: clang-tidy/CMakeLists.txt clang-tidy/ClangTidyForceLinker.h clang-tidy/altera/AlteraTidyModule.cpp clang-tidy/altera/CMakeLists.txt cl

[PATCH] D62731: Add support for options -frounding-math, -ftrapping-math, -ffp-model=, and -ffp-exception-behavior=, : Specify floating point behavior

mibintc closed this revision. mibintc added a comment. Don't know why the commit id didn't get linked when I pushed the change. Here's the closure info: commit af57dbf12e54f3a8ff48534bf1078f4de104c1cd Author: Melanie Blower

[PATCH] D69825: [Clang][Driver] Bypass cc1 process and re-enter driver main

aganea updated this revision to Diff 228729. aganea marked 2 inline comments as done. aganea added a comment. Addressed comments & finished the Linux part. All tests pass. @Meinersbur : Just to show the difference between Windows & Linux, here's some timings for running the tests over the same `

[PATCH] D70094: [clang-tidy] new altera ID dependent backward branch check

ffrankies created this revision. ffrankies added reviewers: alexfh, jdoerfert, hokein, aaron.ballman. ffrankies added projects: clang-tools-extra, clang, LLVM. Herald added subscribers: mgehre, arphaman, xazax.hun, Anastasia, mgorny. This lint check is a part of the FLOCL (FPGA Linters for OpenCL)

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

Anastasia marked an inline comment as done. Anastasia added a comment. In D69938#1737196 , @rjmccall wrote: > 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

[PATCH] D69383: [RISCV] Match GCC `-march`/`-mabi` driver defaults

lenary updated this revision to Diff 228728. lenary added a comment. - Rebase - Update comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69383/new/ https://reviews.llvm.org/D69383 Files: clang/docs/ReleaseNotes.rst clang/lib/Driver/ToolCh

[PATCH] D69383: [RISCV] Match GCC `-march`/`-mabi` driver defaults

lenary added a comment. I think this is just about ready to land, given our discussion on the RISC-V call on 7 November 2019. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69383/new/ https://reviews.llvm.org/D69383 _

[clang-tools-extra] b9213df - [clangd] Fix crash in DefineInline::prepare()

Author: Sam McCall Date: 2019-11-11T19:01:06+01:00 New Revision: b9213dfec4d8ce42d90507c25545564f4a0bbfb8 URL: https://github.com/llvm/llvm-project/commit/b9213dfec4d8ce42d90507c25545564f4a0bbfb8 DIFF: https://github.com/llvm/llvm-project/commit/b9213dfec4d8ce42d90507c25545564f4a0bbfb8.diff LO

[PATCH] D69802: [libTooling] Further simplify `Stencil` type and introduce `MatchComputation`.

This revision was automatically updated to reflect the committed changes. Closed by commit rG489449c28aaa: [libTooling] Further simplify `Stencil` type and introduce `MatchComputation`. (authored by ymandel). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.

[PATCH] D69802: [libTooling] Further simplify `Stencil` type and introduce `MatchComputation`.

ymandel updated this revision to Diff 228720. ymandel added a comment. syntax fixes (to compile) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69802/new/ https://reviews.llvm.org/D69802 Files: clang-tools-extra/clang-tidy/utils/TransformerClangT

[clang-tools-extra] 489449c - [libTooling] Further simplify `Stencil` type and introduce `MatchComputation`.

Author: Yitzhak Mandelbaum Date: 2019-11-11T12:44:15-05:00 New Revision: 489449c28aaa45086d507fbad96826420adf409d URL: https://github.com/llvm/llvm-project/commit/489449c28aaa45086d507fbad96826420adf409d DIFF: https://github.com/llvm/llvm-project/commit/489449c28aaa45086d507fbad96826420adf409d.

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

ABataev added inline comments. Comment at: clang/lib/Sema/SemaOpenMP.cpp:4476 } -if (!ImplicitMaps.empty()) { - CXXScopeSpec MapperIdScopeSpec; - DeclarationNameInfo MapperId; - if (OMPClause *Implicit = ActOnOpenMPMapClause( - llvm::None, llv

[clang-tools-extra] b6cd799 - Fix filename typo in rG8d288a0668a5

Author: Simon Pilgrim Date: 2019-11-11T17:23:21Z New Revision: b6cd799e29a9a5cef4d1d722f267e3b59ef6aaca URL: https://github.com/llvm/llvm-project/commit/b6cd799e29a9a5cef4d1d722f267e3b59ef6aaca DIFF: https://github.com/llvm/llvm-project/commit/b6cd799e29a9a5cef4d1d722f267e3b59ef6aaca.diff LOG:

[PATCH] D69238: Fix clang-tidy readability-redundant-string-init for c++17/c++2a

gribozavr2 accepted this revision. gribozavr2 added a comment. This revision is now accepted and ready to land. Thanks! Do you have commit access or do you need me to commit for you? Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69238/new/ https://r

[PATCH] D69548: Give clang-tidy readability-redundant-string-init a customizable list of string types to fix

gribozavr2 accepted this revision. gribozavr2 added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/docs/clang-tidy/checks/readability-redundant-string-init.rst:29 + +Semicolon-delimited list of base class names to apply this c

[PATCH] D69548: Give clang-tidy readability-redundant-string-init a customizable list of string types to fix

Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/docs/clang-tidy/checks/readability-redundant-string-init.rst:30 +Semicolon-delimited list of base class names to apply this check to. +By default `::std::basic_string` applies to std::string and std::wst

[PATCH] D68682: Clang-tidy fix removals removing all non-blank text from a line should remove the line

gribozavr2 added inline comments. Comment at: clang/include/clang/AST/CommentLexer.h:25 + +/// Requires that BufferPtr point to a newline character (/n or /r). +/// Returns a pointer past the end of any platform newline, i.e. past "\n" and "\r" (everywhere) ===

[clang-tools-extra] 8d288a0 - [clang-tidy] Add bugprone-bad-signal-to-kill-thread check and its alias cert-pos44-c

Author: Abel Kocsis Date: 2019-11-11T17:47:14+01:00 New Revision: 8d288a0668a574863d52784084ff565c89f7366e URL: https://github.com/llvm/llvm-project/commit/8d288a0668a574863d52784084ff565c89f7366e DIFF: https://github.com/llvm/llvm-project/commit/8d288a0668a574863d52784084ff565c89f7366e.diff L

[PATCH] D70087: [ConstExprPreter] String and basic pointer arithmetic.

nand created this revision. nand added reviewers: rsmith, jfb, Bigcheese, dexonsmith. Herald added a project: clang. Herald added a subscriber: cfe-commits. Implemented string constants and the following pointer operations: - AddOffset - SubOffset - Narrow - Expand - LogicalNot To allow for sens

[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:4476 } -if (!ImplicitMaps.empty()) { - CXXScopeSpec MapperIdScopeSpec; - DeclarationNameInfo MapperId; - if (OMPClause *Implicit = ActOnOpenMPMap

[clang-tools-extra] 8cec7e0 - Revert "[clang-tidy] Add bugprone-bad-signal-to-kill-thread checker and alias cert-pos44-c"

Author: Abel Kocsis Date: 2019-11-11T17:34:04+01:00 New Revision: 8cec7e0208f5b65790fd5c73b90d6d35944b07b1 URL: https://github.com/llvm/llvm-project/commit/8cec7e0208f5b65790fd5c73b90d6d35944b07b1 DIFF: https://github.com/llvm/llvm-project/commit/8cec7e0208f5b65790fd5c73b90d6d35944b07b1.diff L

[PATCH] D64305: [clangd] Add path mappings functionality

sammccall added a comment. Thanks, this looks a lot better. Main thing that was unclear to me is the fact that the paths in the mappings are now URI-paths, so "/C:/foo" on windows. This looks like the right idea as it ensures much of the path-mapping code gets to ignore slashes and such. Docs/n

  1   2   >