[clang] d58fd4e - [compiler-rt] Compile assembly files as ASM not C

Author: Petr Hosek Date: 2020-08-20T00:34:59-07:00 New Revision: d58fd4e52197d55bf42ca446c8b0ed31b5c2ec1f URL: https://github.com/llvm/llvm-project/commit/d58fd4e52197d55bf42ca446c8b0ed31b5c2ec1f DIFF: https://github.com/llvm/llvm-project/commit/d58fd4e52197d55bf42ca446c8b0ed31b5c2ec1f.diff LO

[PATCH] D85706: [compiler-rt] Compile assembly files as ASM not C

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rGd58fd4e52197: [compiler-rt] Compile assembly files as ASM not C (authored by phosek). Repository: rG LLVM Github Monore

[PATCH] D81083: [Clang] Allow "vector_size" applied to Booleans

rsandifo-arm added a comment. I'm not qualified to review the CodeGen stuff (or accept the patch, obvs. :-)) but FWIW, here are some comments on the doc and Sema side. It might be good to have more Sema tests for valid and invalid usage, e.g. for which operators are valid and which aren't. Tha

[clang] 1e7ec48 - [AST] Get field size in chars rather than bits in RecordLayoutBuilder.

Author: Bevin Hansson Date: 2020-08-20T10:29:29+02:00 New Revision: 1e7ec4842c1a8b0c686e5674e215012867938a8d URL: https://github.com/llvm/llvm-project/commit/1e7ec4842c1a8b0c686e5674e215012867938a8d DIFF: https://github.com/llvm/llvm-project/commit/1e7ec4842c1a8b0c686e5674e215012867938a8d.diff

[PATCH] D85312: [ADT] Move FixedPoint.h from Clang to LLVM.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG1a995a0af3c4: [ADT] Move FixedPoint.h from Clang to LLVM. (authored by ebevhan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D85191: [AST] Get field size in chars rather than bits in RecordLayoutBuilder.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG1e7ec4842c1a: [AST] Get field size in chars rather than bits in RecordLayoutBuilder. (authored by ebevhan). Repository: rG LLVM Github Monorepo C

[clang] 1a995a0 - [ADT] Move FixedPoint.h from Clang to LLVM.

Author: Bevin Hansson Date: 2020-08-20T10:29:45+02:00 New Revision: 1a995a0af3c4e97c1135b6c9c3a243072ee0463c URL: https://github.com/llvm/llvm-project/commit/1a995a0af3c4e97c1135b6c9c3a243072ee0463c DIFF: https://github.com/llvm/llvm-project/commit/1a995a0af3c4e97c1135b6c9c3a243072ee0463c.diff

[PATCH] D86227: [SyntaxTree] Add support for `MemberExpression`

gribozavr2 added inline comments. Comment at: clang/include/clang/Tooling/Syntax/Nodes.h:180 + MemberExpression_member, + MemberExpression_accessToken, }; Could you order new items in source order? object, access token, member. Comment at: c

[PATCH] D83296: [clang-format] Add a MacroExpander.

klimek added inline comments. Comment at: clang/lib/Format/MacroExpander.cpp:186 +Tok->MacroCtx = MacroContext(MR_ExpandedArg); + pushToken(Tok); +} sammccall wrote: > klimek wrote: > > sammccall wrote: > > > you're pushing here without copying.

[PATCH] D86274: [Constants] Handle FNeg in getWithOperands.

fhahn created this revision. fhahn added reviewers: efriedma, aprantl, arsenm. Herald added subscribers: cfe-commits, hiraditya. Herald added projects: clang, LLVM. fhahn requested review of this revision. Herald added a subscriber: wdng. Currently ConstantExpr::getWithOperands does not handle FNe

[PATCH] D86146: [ARM][BFloat16] Change types of some Arm and AArch64 bf16 intrinsics

miyuki added a comment. In D86146#2226702 , @dmgreen wrote: > Do we need to upgrade the old bfmmla to the new signatures? What is the old bfmmla? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86146/new/ ht

[PATCH] D86227: [SyntaxTree] Add support for `MemberExpression`

eduucaldas updated this revision to Diff 286762. eduucaldas added a comment. Implicit `MemberExpr` generates `id-expression` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86227/new/ https://reviews.llvm.org/D86227 Files: clang/include/clang/Tool

[PATCH] D86146: [ARM][BFloat16] Change types of some Arm and AArch64 bf16 intrinsics

miyuki updated this revision to Diff 286761. miyuki added a comment. Keep the second vbfmlaltq_laneq_f32 test CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86146/new/ https://reviews.llvm.org/D86146 Files: clang/lib/CodeGen/CGBuiltin.cpp clang/test/CodeGen/aarch64-bf16-dotprod-intr

[PATCH] D86146: [ARM][BFloat16] Change types of some Arm and AArch64 bf16 intrinsics

dmgreen added a comment. I mean, do we need upgrade code in llvm/lib/IR/AutoUpgrade.cpp for the new intrinsic forms? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86146/new/ https://reviews.llvm.org/D86146 ___ cfe-commits mailing list cfe-co

[PATCH] D86277: [NFC][compiler-rt] Factor out __mulv[sdt]i3 implementations to .inc file

atrosinenko created this revision. atrosinenko added reviewers: MaskRay, kamleshbhalui, luismarques, efriedma, aykevl. Herald added subscribers: Sanitizers, dberris. Herald added a project: Sanitizers. atrosinenko requested review of this revision. The existing implementations are almost identica

[PATCH] D85736: [Sema][AArch64] Support arm_sve_vector_bits attribute

rsandifo-arm added a comment. LGTM from a spec point of view, but I don't think I should be the one to approve. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85736/new/ https://reviews.llvm.org/D85736 ___ cfe-commits mailing list cfe-commits

[PATCH] D86279: [clangd] Don't crash on `#pragma clang __debug parser_crash`

ArcsinX created this revision. Herald added subscribers: cfe-commits, usaxena95, kadircet, arphaman, jkorous. Herald added a project: clang. ArcsinX requested review of this revision. Herald added subscribers: MaskRay, ilya-biryukov. Currently, clangd crashes when opening a file with `#pragma clan

[PATCH] D83174: Teach AttachPreviousImpl to inherit MSInheritanceAttr attribute

gargvaibhav64 updated this revision to Diff 286786. gargvaibhav64 added a comment. The test is now working properly. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83174/new/ https://reviews.llvm.org/D83174 Files: clang/lib/Serialization/ASTReaderDecl.cpp clang/test/Modules/Inputs/i

[PATCH] D86176: [clang-tidy] readability-simplify-boolean-expr detects negated literals

aaron.ballman added inline comments. Comment at: clang-tools-extra/clang-tidy/readability/SimplifyBooleanExprCheck.cpp:66 + if (const auto *Negated = Result.Nodes.getNodeAs(Id)) { +if (Negated->getOpcode() == UO_LNot && +isa(Negated->getSubExpr()))

[PATCH] D86209: [clang-tidy] run-clang-tidy.py: Fix -allow-enabling-analyzer-alpha-checkers always being passed

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM! Do you need someone to commit on your behalf? If so, please be sure you're fine with the license agreement and let us know what name and email address you would like associ

[PATCH] D86279: [clangd] Don't crash on `#pragma clang __debug parser_crash`

kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land. thanks, lgtm! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86279/new/ https://reviews.llvm.org/D86279

[clang-tools-extra] baeff98 - [clang] When loading preamble from AST file, re-export modules in Sema.

Author: Adam Czachorowski Date: 2020-08-20T14:19:52+02:00 New Revision: baeff989b050e0f63412c52c1b8f9d8f3e91f671 URL: https://github.com/llvm/llvm-project/commit/baeff989b050e0f63412c52c1b8f9d8f3e91f671 DIFF: https://github.com/llvm/llvm-project/commit/baeff989b050e0f63412c52c1b8f9d8f3e91f671.d

[PATCH] D86069: [clang] When loading preamble from AST file, re-export modules in Sema.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGbaeff989b050: [clang] When loading preamble from AST file, re-export modules in Sema. (authored by adamcz). Repository: rG LLVM Github Monorepo C

[clang-tools-extra] 53b9199 - [clangd] Fix crash-bug in preamble indexing when using modules.

Author: Adam Czachorowski Date: 2020-08-20T14:19:52+02:00 New Revision: 53b9199a5cdba8a6e294e1fb183f308ec558db22 URL: https://github.com/llvm/llvm-project/commit/53b9199a5cdba8a6e294e1fb183f308ec558db22 DIFF: https://github.com/llvm/llvm-project/commit/53b9199a5cdba8a6e294e1fb183f308ec558db22.d

[PATCH] D85923: [clangd] Fix crash-bug in preamble indexing when using modules.

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rG53b9199a5cdb: [clangd] Fix crash-bug in preamble indexing when using modules. (authored by adamcz). Repository: rG LLVM

[PATCH] D86282: [Fixed Point] Use FixedPointBuilder to codegen fixed-point IR.

ebevhan created this revision. ebevhan added reviewers: leonardchan, rjmccall. Herald added a subscriber: bjope. Herald added a project: clang. ebevhan requested review of this revision. This changes the methods in CGExprScalar to use FixedPointBuilder to generate IR for fixed-point conversions an

[PATCH] D86229: [X86] Enable constexpr on POPCNT intrinsics (PR31446)

RKSimon added a comment. I think that's everything - I'd prefer to handle the registered target fixes in separate commits. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86229/new/ https://reviews.llvm.org/D86229 __

[PATCH] D85923: [clangd] Fix crash-bug in preamble indexing when using modules.

sammccall added inline comments. Comment at: clang-tools-extra/clangd/unittests/TestTU.cpp:77 + CI.getHeaderSearchOpts().ModuleCachePath = ModuleCachePath.c_str(); + llvm::errs() << "MC: " << ModuleCachePath << "\n"; + llvm::errs().flush(); errs should probabl

[PATCH] D86284: [clangd] Remove useless stderr logging.

adamcz created this revision. Herald added subscribers: cfe-commits, usaxena95, kadircet, arphaman, jkorous. Herald added a project: clang. adamcz requested review of this revision. Herald added subscribers: MaskRay, ilya-biryukov. This was accidentally added in 53b9199a5cdba8a6e294e1fb183f308ec55

[clang-tools-extra] 707138d - [clangd] Remove useless stderr logging.

Author: Adam Czachorowski Date: 2020-08-20T14:52:04+02:00 New Revision: 707138d677861182083b3c6c3b44b76951fd36ef URL: https://github.com/llvm/llvm-project/commit/707138d677861182083b3c6c3b44b76951fd36ef DIFF: https://github.com/llvm/llvm-project/commit/707138d677861182083b3c6c3b44b76951fd36ef.d

[PATCH] D86284: [clangd] Remove useless stderr logging.

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG707138d67786: [clangd] Remove useless stderr logging. (authored by

[PATCH] D86279: [clangd] Don't crash on `#pragma clang __debug parser_crash`

This revision was automatically updated to reflect the committed changes. Closed by commit rG445739826567: [clangd] Don't crash on `#pragma clang __debug parser_crash` (authored by ArcsinX). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86279/new/ h

[clang-tools-extra] 4457398 - [clangd] Don't crash on `#pragma clang __debug parser_crash`

Author: Aleksandr Platonov Date: 2020-08-20T15:53:21+03:00 New Revision: 445739826567e5402b558f2c130d76dc916c82ec URL: https://github.com/llvm/llvm-project/commit/445739826567e5402b558f2c130d76dc916c82ec DIFF: https://github.com/llvm/llvm-project/commit/445739826567e5402b558f2c130d76dc916c82ec.

[PATCH] D86227: [SyntaxTree] Add support for `MemberExpression`

eduucaldas updated this revision to Diff 286799. eduucaldas marked 10 inline comments as done. eduucaldas added a comment. Answering comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86227/new/ https://reviews.llvm.org/D86227 Files: clang/i

[PATCH] D86227: [SyntaxTree] Add support for `MemberExpression`

eduucaldas marked an inline comment as done. eduucaldas added inline comments. Comment at: clang/lib/Tooling/Syntax/BuildTree.cpp:900 + +auto *unqualifiedId = new (allocator()) syntax::UnqualifiedId; +Builder.foldNode(Builder.getRange(S->getMemberLoc(), S->getEndLoc()), -

[PATCH] D86209: [clang-tidy] run-clang-tidy.py: Fix -allow-enabling-analyzer-alpha-checkers always being passed

jspam added a comment. Yes please. I am fine with the license agreement, and please use Joachim Priesner as the commit author. Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86209/new/ https://reviews.llvm.org/D86209 _

[PATCH] D86209: [clang-tidy] run-clang-tidy.py: Fix -allow-enabling-analyzer-alpha-checkers always being passed

aaron.ballman closed this revision. aaron.ballman added a comment. I've commit on your behalf as 03ded5497a2f458b6af054fa7bac0da0240e7b7a , thank you for the patch! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST AC

[clang-tools-extra] 03ded54 - Fix -allow-enabling-analyzer-alpha-checkers always being passed to run-clang-tidy.py

Author: Joachim Priesner Date: 2020-08-20T09:15:29-04:00 New Revision: 03ded5497a2f458b6af054fa7bac0da0240e7b7a URL: https://github.com/llvm/llvm-project/commit/03ded5497a2f458b6af054fa7bac0da0240e7b7a DIFF: https://github.com/llvm/llvm-project/commit/03ded5497a2f458b6af054fa7bac0da0240e7b7a.di

[PATCH] D85706: [compiler-rt] Compile assembly files as ASM not C

teemperor added a comment. This broke Green Dragon's compiler-rt builds and also my local compiler-rt build. See for example here: http://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/23424/console Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/

[clang] adf0b8c - Revert "[compiler-rt] Compile assembly files as ASM not C"

Author: Raphael Isemann Date: 2020-08-20T15:25:22+02:00 New Revision: adf0b8cc70325f027d202139e3ff984c41896b57 URL: https://github.com/llvm/llvm-project/commit/adf0b8cc70325f027d202139e3ff984c41896b57 DIFF: https://github.com/llvm/llvm-project/commit/adf0b8cc70325f027d202139e3ff984c41896b57.dif

[PATCH] D85706: [compiler-rt] Compile assembly files as ASM not C

teemperor added a comment. I reverted in adf0b8cc70325f027d202139e3ff984c41896b57 to get the bots green again . Feel free to ping if you need someone to test the patch on macOS. Repository: rG LLVM Github Monorepo CHANGE

[PATCH] D86227: [SyntaxTree] Add support for `MemberExpression`

gribozavr2 accepted this revision. gribozavr2 added inline comments. This revision is now accepted and ready to land. Comment at: clang/lib/Tooling/Syntax/BuildTree.cpp:911 + S->getEndLoc()), + idExpression, nullptr); + ---

[clang] 2bac004 - Add triples to fixed-point tests which lacked them.

Author: Bevin Hansson Date: 2020-08-20T15:36:15+02:00 New Revision: 2bac004c905dc8db99fd3766678d33aa5a0eec2b URL: https://github.com/llvm/llvm-project/commit/2bac004c905dc8db99fd3766678d33aa5a0eec2b DIFF: https://github.com/llvm/llvm-project/commit/2bac004c905dc8db99fd3766678d33aa5a0eec2b.diff

[PATCH] D83174: Teach AttachPreviousImpl to inherit MSInheritanceAttr attribute

aaron.ballman added inline comments. Comment at: clang/test/Modules/Inputs/inherit-attribute/b.h:3 + +class Foo; + Does this forward declaration impact the test? I can't quite tell whether this exists to test that the attribute is attached to the non-defining re

[PATCH] D86176: [clang-tidy] readability-simplify-boolean-expr detects negated literals

njames93 added inline comments. Comment at: clang-tools-extra/clang-tidy/readability/SimplifyBooleanExprCheck.cpp:66 + if (const auto *Negated = Result.Nodes.getNodeAs(Id)) { +if (Negated->getOpcode() == UO_LNot && +isa(Negated->getSubExpr())) aaron

[PATCH] D86289: [NFC][compiler-rt] Factor out __mulo[sdt]i3 implementations to .inc file

atrosinenko created this revision. atrosinenko added reviewers: MaskRay, kamleshbhalui, luismarques, efriedma, aykevl. Herald added subscribers: Sanitizers, dberris. Herald added a project: Sanitizers. atrosinenko requested review of this revision. The existing implementations are almost identica

[clang] 1ecf120 - [index-while-building] Fix build with -DBUILD_SHARED_LIBS=True

Author: Alex Bradbury Date: 2020-08-20T15:12:56+01:00 New Revision: 1ecf120246e7d3e5c9a9ed1db637914bbf4b5702 URL: https://github.com/llvm/llvm-project/commit/1ecf120246e7d3e5c9a9ed1db637914bbf4b5702 DIFF: https://github.com/llvm/llvm-project/commit/1ecf120246e7d3e5c9a9ed1db637914bbf4b5702.diff

[PATCH] D86089: [flang][driver]Add experimental flang driver and frontend with help screen

richard.barton.arm requested changes to this revision. richard.barton.arm added a comment. A few specific comments to address here as well as the pre-commit linting ones. Comment at: clang/lib/Driver/Driver.cpp:1584 void Driver::PrintVersion(const Compilation &C, raw_ostream &

[PATCH] D86176: [clang-tidy] readability-simplify-boolean-expr detects negated literals

aaron.ballman added inline comments. Comment at: clang-tools-extra/clang-tidy/readability/SimplifyBooleanExprCheck.cpp:66 + if (const auto *Negated = Result.Nodes.getNodeAs(Id)) { +if (Negated->getOpcode() == UO_LNot && +isa(Negated->getSubExpr()))

[PATCH] D86227: [SyntaxTree] Add support for `MemberExpression`

eduucaldas updated this revision to Diff 286811. eduucaldas marked 2 inline comments as done. eduucaldas added a comment. Unify logic for generating `id-expression` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86227/new/ https://reviews.llvm.org/D

[PATCH] D83048: [LiveDebugValues] 3/4 Add Xclang and CodeGen options for using instr-ref variable locations

jmorse updated this revision to Diff 286814. jmorse added a comment. Herald added a subscriber: dang. Add a test for the driver -Xclang option. As far as I understand this, we're just checking that the switch is accepted by the driver, not that it does anything in particular (please correct me i

[PATCH] D85324: [SystemZ][z/OS] Add z/OS Target and define macros

hubert.reinterpretcast added inline comments. Comment at: clang/lib/Basic/Targets/OSTargets.h:736 +MacroBuilder &Builder) const override { +// FIXME: LONG_LONG should not be defined under -std=c89 +Builder.defineMacro("_LONG_LONG"); Mi

[PATCH] D85324: [SystemZ][z/OS] Add z/OS Target and define macros

hubert.reinterpretcast added inline comments. Comment at: clang/test/Preprocessor/init-zos.c:5 +// +// S390X-ZOS-CXX:#define _EXT 1 +// S390X-ZOS-C99:#define _ISOC99_SOURCE 1 Should this be `GNUXX`? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTI

[PATCH] D83088: Introduce CfgTraits abstraction

nhaehnle added a comment. > Based on your description and the DomTree patches, if I understand correctly, > the primary motivation is to facilitate writing CFG-representation-agnostic > algorithms/analyses (e.g., dominators, divergence, convergence analyses), > such that you can later lift the

[clang] ba32915 - [SyntaxTree] Add support for `MemberExpression`

Author: Eduardo Caldas Date: 2020-08-20T14:57:35Z New Revision: ba32915db2ce78256115a9db7b07bb3806e6364a URL: https://github.com/llvm/llvm-project/commit/ba32915db2ce78256115a9db7b07bb3806e6364a DIFF: https://github.com/llvm/llvm-project/commit/ba32915db2ce78256115a9db7b07bb3806e6364a.diff LOG

[PATCH] D86227: [SyntaxTree] Add support for `MemberExpression`

This revision was automatically updated to reflect the committed changes. Closed by commit rGba32915db2ce: [SyntaxTree] Add support for `MemberExpression` (authored by eduucaldas). Changed prior to commit: https://reviews.llvm.org/D86227?vs=286811&id=286820#toc Repository: rG LLVM Github Mon

[clang] a4ef9e8 - [SyntaxTree] Unify logic for generating `id-expression`

Author: Eduardo Caldas Date: 2020-08-20T14:57:35Z New Revision: a4ef9e8643e2f3f8972e19c5b25f4dd81ba03508 URL: https://github.com/llvm/llvm-project/commit/a4ef9e8643e2f3f8972e19c5b25f4dd81ba03508 DIFF: https://github.com/llvm/llvm-project/commit/a4ef9e8643e2f3f8972e19c5b25f4dd81ba03508.diff LOG

[PATCH] D86293: [analyzer] Add modeling of Eq operator in smart ptr

vrnithinkumar created this revision. Herald added subscribers: cfe-commits, steakhal, ASDenysPetrov, martong, Charusso, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun. Herald added a project: clang. vrnithinkumar requested review of this revision.

[PATCH] D86139: [SyntaxTree] Split tests related to Namespace

eduucaldas updated this revision to Diff 286823. eduucaldas marked 2 inline comments as done. eduucaldas added a comment. answer comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86139/new/ https://reviews.llvm.org/D86139 Files: clang/unitt

[PATCH] D86295: [analyzer] Reorder the the layout of MemRegion and cache by hand for optimal size

steakhal created this revision. steakhal added reviewers: NoQ, xazax.hun, Szelethus, vsavchenko, ASDenysPetrov. Herald added subscribers: cfe-commits, martong, Charusso, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware, whisperity. Herald added a project: clang.

[clang] e4e983e - [SyntaxTree] Split tests related to Namespace

Author: Eduardo Caldas Date: 2020-08-20T15:14:56Z New Revision: e4e983e240430b3a0dc92402cc940292bd0d263f URL: https://github.com/llvm/llvm-project/commit/e4e983e240430b3a0dc92402cc940292bd0d263f DIFF: https://github.com/llvm/llvm-project/commit/e4e983e240430b3a0dc92402cc940292bd0d263f.diff LOG

[PATCH] D86139: [SyntaxTree] Split tests related to Namespace

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGe4e983e24043: [SyntaxTree] Split tests related to Namespace (authored by eduucaldas). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTI

[PATCH] D86065: [SVE] Make ElementCount members private

david-arm added a comment. Hi @ctetreau, I agree with @efriedma that keeping the two classes distinct for now seems best. The reason is I spent quite a lot of time trying to unify these classes already and I hit a stumbling block - TypeSize has the ugly uint64_t() cast operator, which makes uni

[PATCH] D86298: [SyntaxTree] Add support for `this`

eduucaldas created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. eduucaldas requested review of this revision. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D86298 Files: clang/include/clang/Tooling/Syntax/Nodes.h clang/lib/Tooling/Syn

[PATCH] D86299: [compiler-rt][builtins] Factor out some common bit manipulations

atrosinenko created this revision. atrosinenko added reviewers: howard.hinnant, jfb, vsk, lebedev.ri. Herald added subscribers: Sanitizers, dexonsmith, dberris. Herald added a project: Sanitizers. atrosinenko requested review of this revision. This patch incapsulates some commonly used hacks in a

[PATCH] D84414: [RISCV] Support Shadow Call Stack

lenary added a comment. In D84414#2186257 , @jrtc27 wrote: > There is a possibly-compelling argument against using x18: RV32E only gives > x0-x15, so would not be able to support the current implementation. We discussed this on the RISC-V LLVM Sync-up (b

[PATCH] D86229: [X86] Enable constexpr on POPCNT intrinsics (PR31446)

RKSimon updated this revision to Diff 286835. RKSimon added a comment. Added release note entry - this will need to revised as we add more intrinsics, so is just an initial placeholder. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86229/new/ http

[PATCH] D86229: [X86] Enable constexpr on POPCNT intrinsics (PR31446)

RKSimon updated this revision to Diff 286837. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86229/new/ https://reviews.llvm.org/D86229 Files: clang/docs/ReleaseNotes.rst clang/lib/Headers/popcntintrin.h clang/test/CodeGen/popcnt-builtins.c I

[PATCH] D81083: [Clang] Allow "vector_size" applied to Booleans

simoll planned changes to this revision. simoll marked 6 inline comments as done. simoll added inline comments. Comment at: clang/docs/LanguageExtensions.rst:492 + +The memory representation of a boolean vector is the smallest fitting +power-of-two integer. The alignment is the a

[PATCH] D85031: [builtins] Unify the softfloat division implementation

sepavloff added inline comments. Comment at: compiler-rt/lib/builtins/fp_div_impl.inc:99 + // 0 < x0(b) < 1 + // abs(x0(b) - 1/b) <= 3/4 - 1/sqrt(2) + This estimation is absent from the original comment. Do you have reference where it came from? Also the o

[PATCH] D86223: [analyzer][z3] Use more elaborate z3 variable names in debug build

xazax.hun added a comment. In D86223#2227353 , @steakhal wrote: > Eh, I'm not sure if it worth it to put these into virtual functions - as > conditionally overriding functions is not really a good idea. I am not sure I follow. What do you mean by conditi

[PATCH] D86295: [analyzer] Reorder the layout of MemRegion and cache by hand for optimal size

xazax.hun added a comment. MemRegion is a popular class to instantiate in the analyzer so it looks good to me. But unless you add a comment one will probably replace the offset with an optional as the part of a refactoring. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION htt

[PATCH] D86229: [X86] Enable constexpr on POPCNT intrinsics (PR31446)

craig.topper added a comment. Should we test the intrinsics in a constexpr context? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86229/new/ https://reviews.llvm.org/D86229 ___ cfe-commits mailing list c

[PATCH] D77925: Revert "[TLI] Per-function fveclib for math library used for vectorization"

wenlei added a comment. In D77925#2220177 , @wenlei wrote: > In D77925#2220169 , @mehdi_amini > wrote: > >> I rather have a non closed list of veclib: if you just have a map keyed by >> string instead of an enum i

[PATCH] D86223: [analyzer][z3] Use more elaborate z3 variable names in debug build

steakhal added a comment. In D86223#2228731 , @xazax.hun wrote: > In D86223#2227353 , @steakhal wrote: > >> Eh, I'm not sure if it worth it to put these into virtual functions - as >> conditionally overriding funct

[PATCH] D78902: [Driver] Add output file to properties of Command

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

[PATCH] D78899: [Driver] Add callback to Command execution

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

[PATCH] D86295: [analyzer] Reorder the layout of MemRegion and cache by hand for optimal size

steakhal added a comment. In D86295#2228745 , @xazax.hun wrote: > But unless you add a comment one will probably replace the offset with an > optional as the part of a refactoring. Sure, I will leave a note. I was thinking using a PointerIntUnion though

[PATCH] D86223: [analyzer][z3] Use more elaborate z3 variable names in debug build

xazax.hun added a comment. In D86223#2228855 , @steakhal wrote: > I wanted to conditionally, aka. in debug configuration override the base > implementation of the SymbolData::getKindStr I see. Yeah, that does not make much sense. I was thinking about alw

[PATCH] D85692: python bindings: fix DeprecationWarning

nickdesaulniers added a comment. In D85692#2227531 , @jdenny wrote: > Does this mean we officially no longer support python 2, which this change > breaks? Send a patch to support both and I'll approve it. Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D86308: Reland [compiler-rt] Compile assembly files as ASM not C

tambre created this revision. tambre added reviewers: phosek, teemperor. Herald added subscribers: Sanitizers, cfe-commits, dexonsmith, kristof.beyls, mgorny, dberris. Herald added projects: clang, Sanitizers. tambre requested review of this revision. It isn't very wise to pass an assembly file t

[PATCH] D85031: [builtins] Unify the softfloat division implementation

atrosinenko added a comment. //Some general context:// The final goal was to have an explanation why this particular number of iteration (3, 4 or 5, depending on type) are enough for any `a` and `b` passed as input arguments taking into account errors due to particular finite precision computat

[PATCH] D85324: [SystemZ][z/OS] Add z/OS Target and define macros

abhina.sreeskantharajan updated this revision to Diff 286853. abhina.sreeskantharajan marked an inline comment as done. abhina.sreeskantharajan added a comment. Thanks Hubert, I updated the comments, and also the check-prefix to your suggestion. Repository: rG LLVM Github Monorepo CHANGES SI

[PATCH] D85324: [SystemZ][z/OS] Add z/OS Target and define macros

abhina.sreeskantharajan marked 3 inline comments as done. abhina.sreeskantharajan added inline comments. Comment at: clang/lib/Basic/Targets/OSTargets.h:743 +Builder.defineMacro("__BFP__"); +Builder.defineMacro("__BOOL__"); +Builder.defineMacro("__LONGNAME__"); --

[PATCH] D86308: Reland [compiler-rt] Compile assembly files as ASM not C

tambre added a comment. I'm pretty sure `add_asm_sources()` has nothing to do. The ASM language is enabled by compiler-rt anyway and CMake can recognize the files as assembly anyway. @teemperor Could you give this a try? If this doesn't work, could you upload the Ninja file for me to investiga

[PATCH] D85692: python bindings: fix DeprecationWarning

jdenny added a comment. In D85692#2228860 , @nickdesaulniers wrote: > In D85692#2227531 , @jdenny wrote: > >> Does this mean we officially no longer support python 2, which this change >> breaks? > > Send a patch

[PATCH] D86229: [X86] Enable constexpr on POPCNT intrinsics (PR31446)

RKSimon updated this revision to Diff 286861. RKSimon added a comment. Added constexpr tests - I went for the -ve array size approach as thats the approach in clang\test\Sema\constant-builtins-2.c etc. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D

[PATCH] D86310: [X86] Align i128 to 16 bytes in x86-64 datalayout

craig.topper created this revision. craig.topper added reviewers: efriedma, echristo, RKSimon, spatel. Herald added subscribers: nikic, dexonsmith, steven_wu, javed.absar, hiraditya, dschuff. Herald added a project: LLVM. craig.topper requested review of this revision. This is an attempt at reboo

[PATCH] D82994: [RFC] Instrumenting Clang/LLVM with Perfetto

Nathan-Huckleberry updated this revision to Diff 286866. Nathan-Huckleberry added a comment. - Attach translation unit names to trace processes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82994/new/ https://reviews.llvm.org/D82994 Files: clang

[PATCH] D86293: [analyzer] Add modeling of Eq operator in smart ptr

vrnithinkumar updated this revision to Diff 286868. vrnithinkumar added a comment. - Add assignment to nullptr case Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86293/new/ https://reviews.llvm.org/D86293 Files: clang/lib/StaticAnalyzer/Checkers

[PATCH] D86229: [X86] Enable constexpr on POPCNT intrinsics (PR31446)

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/D86229/new/ https://reviews.llvm.org/D86229 _

[PATCH] D85324: [SystemZ][z/OS] Add z/OS Target and define macros

MaskRay added inline comments. Comment at: clang/lib/Basic/Targets/OSTargets.h:743 +Builder.defineMacro("__BFP__"); +// FIXME: __BOOL__ should be defined under strict -std=c89. +Builder.defineMacro("__BOOL__"); What is strict -std=c89? `!Opts.C99` ?

[PATCH] D86308: Reland [compiler-rt] Compile assembly files as ASM not C

phosek added a comment. In D86308#2228917 , @tambre wrote: > I'm pretty sure `add_asm_sources()` has nothing to do. The ASM language is > enabled by compiler-rt anyway and CMake can recognize the files as assembly > anyway. Can we remove that function a

[PATCH] D86308: Reland [compiler-rt] Compile assembly files as ASM not C

mstorsjo added a comment. Just FWIW, a similar change was made in libunwind earlier (c48974ffd7d1676f79d39d3b1e70f07d3a5e2e44 ), which then required workarounds for cmake issues on both mingw and macos (see b780df052dd2b246a7

[PATCH] D86089: [flang][driver]Add experimental flang driver and frontend with help screen

sameeranjoshi added a comment. Thanks for the work. A couple of comments on `clang/` related changes: An `out-of-tree` build with this patch fails for me: Here's what I did: I initially used `ENABLE_PROJECTS="clang;mlir"` to build `llvm-project`, I didn't build `flang` during this run. Then I p

[clang] cff0db0 - [X86] Enable constexpr on POPCNT intrinsics (PR31446)

Author: Simon Pilgrim Date: 2020-08-20T21:38:04+01:00 New Revision: cff0db08761f310dfebb5b41b307d2c12bda85fc URL: https://github.com/llvm/llvm-project/commit/cff0db08761f310dfebb5b41b307d2c12bda85fc DIFF: https://github.com/llvm/llvm-project/commit/cff0db08761f310dfebb5b41b307d2c12bda85fc.diff

[PATCH] D86229: [X86] Enable constexpr on POPCNT intrinsics (PR31446)

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGcff0db08761f: [X86] Enable constexpr on POPCNT intrinsics (PR31446) (authored by RKSimon). Changed prior to commit: https://reviews.llvm.org/D8622

[PATCH] D86154: AMDGPU: Add llvm.amdgcn.{read,readfirst,write}lane2 intrinsics with type overloads

nhaehnle updated this revision to Diff 286898. nhaehnle added a comment. Don't duplicate the intrinsics. Rely on D86317 to reduce the pain of this change caused to downstream users. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://revi

[PATCH] D86154: AMDGPU: Add llvm.amdgcn.{read,readfirst,write}lane2 intrinsics with type overloads

nhaehnle added a comment. In D86154#2224272 , @arsenm wrote: > In D86154#2224270 , @nhaehnle wrote: > >> Note that part of my motivation here over D84639 >> is to support more gene

[PATCH] D85528: [analyzer] Fix cast evaluation on scoped enums in ExprEngine

steakhal added a comment. It would be nice to have this fix in clang11. Do you think it's qualified for it? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85528/new/ https://reviews.llvm.org/D85528 ___ cf

  1   2   >