[PATCH] D85612: [Sema] Use proper integral cast for an enumerate with a fixed bool type

rsmith added inline comments. Comment at: clang/test/AST/ast-dump-enum-bool.cpp:1 +// RUN: %clang_cc1 -std=c++20 -triple x86_64-unknown-unknown -fsyntax-only -ast-dump %s | FileCheck %s + I don't think we really need a dedicated AST test for this. Such tests cre

[PATCH] D73865: [CodeGenModule] Assume dso_local for -fpic -fno-semantic-interposition

MaskRay abandoned this revision. MaskRay added a comment. Herald added subscribers: aaron.ballman, sstefan1. Landed with a change that only changes the explicit -fno-semantic-interposition behavior, so there was very little test change. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[clang] 4b59dc7 - [Diagnostics] Ignore structs and long text for -Wstring-concatenation

Author: Dávid Bolvanský Date: 2020-08-11T00:49:58+02:00 New Revision: 4b59dc77dc473bba849c1b08f3a1ab7be5733ad1 URL: https://github.com/llvm/llvm-project/commit/4b59dc77dc473bba849c1b08f3a1ab7be5733ad1 DIFF: https://github.com/llvm/llvm-project/commit/4b59dc77dc473bba849c1b08f3a1ab7be5733ad1.dif

Re: [PATCH] D85545: [Diagnostics] Diagnose missing comma in string array initialization

Pushed fix. It should not warn for structs or long texts. ut 11. 8. 2020 o 0:34 Arthur Eubanks via Phabricator napísal(a): > > aeubanks added a comment. > > In D85545#2208266 , @arthur.j.odwyer > wrote: > > > To decrease the number of false-positives, you

[clang] 09517a9 - Add regression test from PR46487.

Author: Richard Smith Date: 2020-08-10T15:55:27-07:00 New Revision: 09517a90682f05f13dca7d976aeaa5437beeb8f3 URL: https://github.com/llvm/llvm-project/commit/09517a90682f05f13dca7d976aeaa5437beeb8f3 DIFF: https://github.com/llvm/llvm-project/commit/09517a90682f05f13dca7d976aeaa5437beeb8f3.diff

[PATCH] D85695: [OpenMP] split execution of a long test into smaller parts.

tra created this revision. tra added reviewers: ABataev, jdoerfert. Herald added subscribers: sanjoy.google, bixia, guansong, yaxunl. Herald added a project: clang. tra requested review of this revision. Herald added a subscriber: sstefan1. This test is bottlenecked by heavy regex use (~0.6s per F

[PATCH] D85551: [OpenMP] Split OpenMP/target_map_codegen test [NFC]

tra added inline comments. Comment at: clang/test/OpenMP/target_map_codegen_18.cpp:5 + +///==/// +// RUN: %clang_cc1 -DUSE -DCK19 -verify -fopenmp -fopenmp-targets=powerpc64le-ibm-linux-gnu -x c++ -triple p

[PATCH] D84068: AMDGPU/clang: Search resource directory for device libraries

arsenm added a comment. In D84068#2208132 , @tra wrote: > In D84068#2204713 , @arsenm wrote: > >>> If we ship them with clang, who/where/how builds them? >>> If they come from ROCm packages, how would those packages

[PATCH] D85685: [WIP] Support dwarf fission for wasm object files

dschuff updated this revision to Diff 284522. dschuff added a comment. Fix bad diff Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85685/new/ https://reviews.llvm.org/D85685 Files: clang/lib/Driver/ToolChains/Clang.cpp llvm/include/llvm/MC/MCWa

[PATCH] D85695: [OpenMP] split execution of a long test into smaller parts.

tra added a comment. The test is no longer sticking out: https://gist.github.com/Artem-B/d0b05c2e98a49158c02de23f7f4f0279 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85695/new/ https://reviews.llvm.org/D85695 ___

[PATCH] D85686: [CUDA][HIP] Do not externalize implicit constant static variable

This revision was automatically updated to reflect the committed changes. yaxunl marked an inline comment as done. Closed by commit rGfb04d7b4a698: [CUDA][HIP] Do not externalize implicit constant static variable (authored by yaxunl). Herald added a project: clang. Changed prior to commit: http

[clang] fb04d7b - [CUDA][HIP] Do not externalize implicit constant static variable

Author: Yaxun (Sam) Liu Date: 2020-08-10T19:02:49-04:00 New Revision: fb04d7b4a69831f6b999b1776da738557b108e0d URL: https://github.com/llvm/llvm-project/commit/fb04d7b4a69831f6b999b1776da738557b108e0d DIFF: https://github.com/llvm/llvm-project/commit/fb04d7b4a69831f6b999b1776da738557b108e0d.dif

[PATCH] D85619: [clang][OpenMP][OMPBuilder] Use OMPBuilder to CG `omp single`

kiranchandramohan added a comment. Before I start, Why is this change all new code and no modification or deletion of existing Clang functionality for omp single/copyprivate? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85619/new/ https://reviews

[PATCH] D84068: AMDGPU/clang: Search resource directory for device libraries

yaxunl added a comment. In D84068#2208394 , @arsenm wrote: >> > > Long term, I'd rather aim for merging rocm-device-libs into libclc and making > it an llvm project. They're largely forks of the same original sources from > about 5 years ago, and it's a

[PATCH] D85696: [AST] add parenthesis locations for IfStmt and SwitchStmt

zequanwu created this revision. zequanwu added a reviewer: rsmith. Herald added subscribers: cfe-commits, martong. Herald added a reviewer: shafik. Herald added a project: clang. zequanwu requested review of this revision. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D85696 Fil

[clang] 0fd3d37 - Improve diagnostic for an expression that's not constant because it uses

Author: Richard Smith Date: 2020-08-10T17:03:19-07:00 New Revision: 0fd3d379e26da3cb02b6d960198e2efbc5190650 URL: https://github.com/llvm/llvm-project/commit/0fd3d379e26da3cb02b6d960198e2efbc5190650 DIFF: https://github.com/llvm/llvm-project/commit/0fd3d379e26da3cb02b6d960198e2efbc5190650.diff

[PATCH] D85696: [AST] add parenthesis locations for IfStmt and SwitchStmt

zequanwu updated this revision to Diff 284532. zequanwu added a comment. Forgot changes on ASTReaderStmt.cpp, added now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85696/new/ https://reviews.llvm.org/D85696 Files: clang/include/clang/AST/Stmt

[PATCH] D85697: [clang-tidy] Add cppcoreguidelines-prefer-scoped-enums-over-unscoped

janosbenjaminantal created this revision. janosbenjaminantal added reviewers: alexfh, njames93, xazax.hun, Eugene.Zelenko. Herald added subscribers: cfe-commits, aaron.ballman, rnkovacs, kbarton, mgorny, nemanjai. Herald added a project: clang. janosbenjaminantal requested review of this revision.

[PATCH] D85685: [WIP] Support dwarf fission for wasm object files

aardappel accepted this revision. aardappel added a comment. This revision is now accepted and ready to land. Nice, fairly unintrusive actually. If you desperately wanted to fix the need for changing `W` dynamically, you could instead make it allocate a second `WasmObjectWriter` to write the `dw

[PATCH] D85697: [clang-tidy] Add cppcoreguidelines-prefer-scoped-enums-over-unscoped

Eugene.Zelenko added a comment. Please mention new check in Release Notes. Comment at: clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines-prefer-scoped-enums-over-unscoped.rst:22 + +After the fix is applied, the enum will become: + Please enclose enum

[PATCH] D85385: [X86][FPEnv] Teach X86 mask compare intrinsics to respect strict FP semantics.

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

[PATCH] D85619: [clang][OpenMP][OMPBuilder] Use OMPBuilder to CG `omp single`

fghanim added a comment. In D85619#2208425 , @kiranchandramohan wrote: > Before I start, Why is this change all new code and no modification or > deletion of existing Clang functionality for omp single/copyprivate? Because the OMPBuilder currently is no

[PATCH] D85695: [OpenMP] split execution of a long test into smaller parts.

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/D85695/new/ https://reviews.llvm.org/D85695 ___ c

[PATCH] D85385: [X86][FPEnv] Teach X86 mask compare intrinsics to respect strict FP semantics.

craig.topper added inline comments. Comment at: clang/test/CodeGen/avx512f-builtins-constrained-cmp.c:793 // CHECK-LABEL: test_mm512_mask_cmp_pd_mask_true_us - // CHECK: [[CMP:%.*]] = call <8 x i1> @llvm.x86.avx512.cmp.pd.512(<8 x double> %{{.*}}, <8 x double> %{{.*}}, i32 3

[PATCH] D79773: [clang-format] Improve clang-formats handling of concepts

JohelEGP added a comment. I found another case. See how everything after the //requires-clause// is indented for constructors with just the constructor's name (it works otherwise, maybe because it looks like a function). class [[nodiscard]] data_t { public: template requi

[PATCH] D85099: [UpdateTestChecks] Match unnamed values like "@[0-9]+" and "![0-9]+"

jdoerfert updated this revision to Diff 284537. jdoerfert added a comment. Herald added a project: clang. Herald added a subscriber: cfe-commits. Update clang tests Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85099/new/ https://reviews.llvm.org/D

[PATCH] D85699: PR47099: Split out of .

rsmith created this revision. rsmith added reviewers: hans, BillyONeal. Herald added subscribers: cfe-commits, jfb, mgorny. Herald added a project: clang. rsmith requested review of this revision. This aims to be compatible with Visual C++'s , but not exactly match it. The contents of are a subse

[PATCH] D85099: [UpdateTestChecks] Match unnamed values like "@[0-9]+" and "![0-9]+"

MaskRay added a comment. Mostly looks good. Comment at: llvm/utils/UpdateTestChecks/common.py:309 + +nameless_values = [] +# Description of the different "unnamed" values we match in the IR, e.g., ```lang=python nameless_values = [ NamelessValue(r'TMP', r'

[PATCH] D85574: [Sema] Fix missing warning on initializer lists on field initializers with overloaded operators

rtrieu added inline comments. Comment at: clang/lib/Sema/SemaDeclCXX.cpp:3584-3585 if (!isa(Base->IgnoreParenImpCasts())) return; Does the warning work if it was changed to be "Visit(Base);" before the return here instead of the change above?

[PATCH] D85601: Fixes an assertion when IntRange processes a throw expression

rsmith added inline comments. Comment at: clang/lib/Sema/SemaChecking.cpp:10164 const BuiltinType *BT = cast(T); -assert(BT->isInteger()); +if (!BT->isInteger()) { + // This can happen in a conditional expression with a throw statement Mordante

[clang] ed66df6 - test commit

Author: Albion Fung Date: 2020-08-10T21:18:36-04:00 New Revision: ed66df6705a9dffc36fecc3468cc3ab430849182 URL: https://github.com/llvm/llvm-project/commit/ed66df6705a9dffc36fecc3468cc3ab430849182 DIFF: https://github.com/llvm/llvm-project/commit/ed66df6705a9dffc36fecc3468cc3ab430849182.diff L

[PATCH] D85696: [AST] add parenthesis locations for IfStmt and SwitchStmt

rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Only relatively trivial comments; please feel free to commit once they're addressed. Comment at: clang/include/clang/Sema/Sema.h:4382-4393 + StmtResult ActOnIfStmt(SourceLo

[PATCH] D83088: Introduce CfgTraits abstraction

dblaikie added a comment. This seems like a strange hybrid between a static-polymorphism (with traits) and dynamic polymorphism (with base classes/virtual functions). Could this more readily be just one or the other? (sounds like you're leaning towards dynamic polymorphism) Repository: rG L

[PATCH] D85099: [UpdateTestChecks] Match unnamed values like "@[0-9]+" and "![0-9]+"

jdoerfert updated this revision to Diff 284543. jdoerfert marked an inline comment as done. jdoerfert added a comment. added @MaskRays's tweaks Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85099/new/ https://reviews.llvm.org/D85099 Files: clang

[PATCH] D85099: [UpdateTestChecks] Match unnamed values like "@[0-9]+" and "![0-9]+"

jdoerfert added a comment. (I'm going to sneak in `!range !` as well) Comment at: llvm/utils/UpdateTestChecks/common.py:330 +IR_VALUE_REGEXP_STRING += nameless_value.ir_prefix + r'(' + nameless_value.ir_regexp + r')' +IR_VALUE_REGEXP_SUFFIX = r'([,\s\(\)]|\Z)' +IR_VALUE_RE

[PATCH] D85385: [X86][FPEnv] Teach X86 mask compare intrinsics to respect strict FP semantics.

pengfei added inline comments. Comment at: clang/test/CodeGen/avx512f-builtins-constrained-cmp.c:793 // CHECK-LABEL: test_mm512_mask_cmp_pd_mask_true_us - // CHECK: [[CMP:%.*]] = call <8 x i1> @llvm.x86.avx512.cmp.pd.512(<8 x double> %{{.*}}, <8 x double> %{{.*}}, i32 31, i3

[PATCH] D85574: [Sema] Fix missing warning on initializer lists on field initializers with overloaded operators

zequanwu updated this revision to Diff 284551. zequanwu marked an inline comment as done. zequanwu added a comment. Address comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85574/new/ https://reviews.llvm.org/D85574 Files: clang/lib/Sema/S

[PATCH] D85574: [Sema] Fix missing warning on initializer lists on field initializers with overloaded operators

zequanwu added inline comments. Comment at: clang/lib/Sema/SemaDeclCXX.cpp:3584-3585 if (!isa(Base->IgnoreParenImpCasts())) return; rtrieu wrote: > Does the warning work if it was changed to be "Visit(Base);" before the > return here instead

[PATCH] D85385: [X86][FPEnv] Teach X86 mask compare intrinsics to respect strict FP semantics.

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

[PATCH] D85574: [Sema] Fix missing warning on initializer lists on field initializers with overloaded operators

rtrieu accepted this revision. rtrieu 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/D85574/new/ https://reviews.llvm.org/D85574 ___ c

[PATCH] D85696: [AST] add parenthesis locations for IfStmt and SwitchStmt

zequanwu updated this revision to Diff 284557. zequanwu marked 3 inline comments as done. zequanwu added a comment. Address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85696/new/ https://reviews.llvm.org/D85696 Files: clang/include/c

[clang] 94c6cea - [AST] add parenthesis locations for IfStmt and SwitchStmt

Author: Zequan Wu Date: 2020-08-10T19:19:51-07:00 New Revision: 94c6ceab539efe26a1707124e803f444139c1b12 URL: https://github.com/llvm/llvm-project/commit/94c6ceab539efe26a1707124e803f444139c1b12 DIFF: https://github.com/llvm/llvm-project/commit/94c6ceab539efe26a1707124e803f444139c1b12.diff LOG

[PATCH] D85696: [AST] add parenthesis locations for IfStmt and SwitchStmt

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG94c6ceab539e: [AST] add parenthesis locations for IfStmt and SwitchStmt (authored by zequanwu). Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D85545: [Diagnostics] Diagnose missing comma in string array initialization

aeubanks added a comment. After not warning on structs, this did find a real missing comma in the Chromium codebase, so thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85545/new/ https://reviews.llvm.org/D85545 _

[clang] 4aaa977 - [Sema] Fix missing warning on initializer lists on field initializers with overloaded operators

Author: Zequan Wu Date: 2020-08-10T19:32:59-07:00 New Revision: 4aaa97700377a811dbcaf88a81ec068a5b517a4d URL: https://github.com/llvm/llvm-project/commit/4aaa97700377a811dbcaf88a81ec068a5b517a4d DIFF: https://github.com/llvm/llvm-project/commit/4aaa97700377a811dbcaf88a81ec068a5b517a4d.diff LOG

[PATCH] D85574: [Sema] Fix missing warning on initializer lists on field initializers with overloaded operators

This revision was automatically updated to reflect the committed changes. Closed by commit rG4aaa97700377: [Sema] Fix missing warning on initializer lists on field initializers with… (authored by zequanwu). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.or

[PATCH] D85431: [analyzer] Implement a new checker ThreadPrimitivesChecker

NoQ added a comment. Umm, why don't you extend `PthreadLockChecker` instead? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85431/new/ https://reviews.llvm.org/D85431 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.o

[PATCH] D84520: [Analyzer] Improve invalid dereference bug reporting in DereferenceChecker.

NoQ added a comment. In D84520#2206077 , @balazske wrote: > Do the null pointer and invalid pointer dereference belong to the same > checker, that is called //NullDereference//? Yup. And that's bad. Note that the only reason to have checker names is to

[PATCH] D84600: [Analyzer] Support note tags for smart ptr checker

NoQ accepted this revision. NoQ added a comment. Let's land this! Damn, we've learned //a lot// from this patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84600/new/ https://reviews.llvm.org/D84600 _

[PATCH] D37809: [analyzer] PthreadLock: Refactor, use PostCall API. NFC.

NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Herald added subscribers: steakhal, ASDenysPetrov, martong, Charusso, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, rnkovacs, szepet, baloghadamsoftware. This patch has already landed as rG152bc7ff

[PATCH] D85576: [clang][Fuchsia] Add relative-vtables multilib

leonardchan updated this revision to Diff 284573. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85576/new/ https://reviews.llvm.org/D85576 Files: clang/cmake/caches/Fuchsia-stage2.cmake clang/lib/Driver/ToolChains/Fuchsia.cpp clang/test/Drive

[clang] 5fe1713 - [Sparc] Define __GCC_HAVE_SYNC_COMPARE_AND_SWAP macros on SPARCv9

Author: Brad Smith Date: 2020-08-11T00:04:24-04:00 New Revision: 5fe171321c018a811debc306a776dbdf27a306dd URL: https://github.com/llvm/llvm-project/commit/5fe171321c018a811debc306a776dbdf27a306dd DIFF: https://github.com/llvm/llvm-project/commit/5fe171321c018a811debc306a776dbdf27a306dd.diff LO

Buildbot numbers for the week of 07/26/2020 - 08/01/2020

Hello everyone, Below are some buildbot numbers for the week of 07/26/2020 - 08/01/2020. Please see the same data in attached csv files: The longest time each builder was red during the week; "Status change ratio" by active builder (percent of builds that changed the builder status from green to

Buildbot numbers for the week of 07/19/2020 - 07/25/2020

Hello everyone, Below are some buildbot numbers for the week of 07/19/2020 - 07/25/2020. Please see the same data in attached csv files: The longest time each builder was red during the week; "Status change ratio" by active builder (percent of builds that changed the builder status from green to

Buildbot numbers for the week of 08/02/2020 - 08/08/2020

Hello everyone, Below are some buildbot numbers for the last week of 08/02/2020 - 08/08/2020. Please see the same data in attached csv files: The longest time each builder was red during the week; "Status change ratio" by active builder (percent of builds that changed the builder status from gre

[PATCH] D85645: [AST] Fix the CXXFoldExpr source range when parentheses range is invalid.

nridge added inline comments. Comment at: clang/include/clang/AST/ExprCXX.h:4587 + return RParenLoc; +return getPattern()->getEndLoc(); + } Wouldn't `EllipsisLoc` be more accurate? Comment at: clang/test/AST/ast-dump-concepts.cpp:16 +

[PATCH] D85099: [UpdateTestChecks] Match unnamed values like "@[0-9]+" and "![0-9]+"

MaskRay accepted this revision. MaskRay added a comment. This revision is now accepted and ready to land. LGTM. I don't have further comments. Worth waiting for other opinions. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85099/new/ https://review

[PATCH] D85621: [clang] Allow DynTypedNode to store a TemplateArgumentLoc

nridge added a comment. In D85621#2205944 , @njames93 wrote: > In D85621#2205856 , @nridge wrote: > >> In D85621#2205803 , @hokein wrote: >> >>> Please run `clang/docs/tools/

[PATCH] D83536: [clangd] Index refs to main-file symbols as well

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

[PATCH] D84520: [Analyzer] Improve invalid dereference bug reporting in DereferenceChecker.

NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Also thanks, let's land! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84520/new/ https://reviews.llvm.org/D84520 ___

[PATCH] D59615: [AArch64] When creating SISD intrinsic calls widen scalar args into a zero vectors, not undef

aemerson added a comment. Herald added a subscriber: danielkiss. Does anyone object to this? I'd like to get it off my review dashboard one way or the other. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59615/new/ https://reviews.llvm.org/D59615

[PATCH] D59615: [AArch64] When creating SISD intrinsic calls widen scalar args into a zero vectors, not undef

efriedma added a comment. I see two issues here: 1. We're still generating the wrong instruction. 2. The intrinsic is marked readnone, so any code that depends on whether it sets saturation flags is likely broken anyway. Given the layers of wrongness here, this seems like a marginal improvement

[PATCH] D84520: [Analyzer] Improve invalid dereference bug reporting in DereferenceChecker.

balazske updated this revision to Diff 284588. balazske marked an inline comment as done. balazske added a comment. Rebase (bug category is LogicError). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84520/new/ https://reviews.llvm.org/D84520 Files

<    1   2   3