[PATCH] D87822: [FPEnv] Evaluate constant expressions under non-default rounding modes

sepavloff added inline comments. Comment at: clang/lib/AST/ExprConstant.cpp:2438 +/// Check if the given evaluation result is allowed for constant evaluation. +static bool checkFloatingPointResult(EvalInfo , const Expr *E, + APFloat::opStatus

[PATCH] D87822: [FPEnv] Evaluate constant expressions under non-default rounding modes

sepavloff updated this revision to Diff 294230. sepavloff added a comment. Added diagnostic message Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87822/new/ https://reviews.llvm.org/D87822 Files: clang/include/clang/AST/Expr.h

[PATCH] D88278: [PowerPC] Add builtins for xvtdiv(dp|sp) and xvtsqrt(dp|sp).

Esme created this revision. Esme added reviewers: steven.zhang, masoud.ataei, shchenz, jsji, qiucf. Herald added subscribers: llvm-commits, cfe-commits, kbarton, hiraditya, nemanjai. Herald added projects: clang, LLVM. Esme requested review of this revision. This patch implements the builtins

[clang] 7db7a35 - Fix uninitialized XRayArg

Author: Ian Levesque Date: 2020-09-25T00:20:36-04:00 New Revision: 7db7a355453887906d12ffb67df8fbaa5e9e873d URL: https://github.com/llvm/llvm-project/commit/7db7a355453887906d12ffb67df8fbaa5e9e873d DIFF: https://github.com/llvm/llvm-project/commit/7db7a355453887906d12ffb67df8fbaa5e9e873d.diff

[PATCH] D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata

rjmccall added inline comments. Comment at: clang/lib/CodeGen/CGObjC.cpp:490 + llvm::UniqueVector FoundProtocols; + std::set DeclaredProtocols; + You should use llvm::DenseSet for this. But in general there are more sets here than I think you really need,

[PATCH] D88130: [PPC] [AIX] Implement calling convention IR for C99 complex types on AIX

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

[PATCH] D88105: [NFC] [PPC] Add PowerPC expected IR tests for C99 complex

This revision was automatically updated to reflect the committed changes. Closed by commit rG64b8a633a872: [NFC] [PPC] Add PowerPC expected IR tests for C99 complex (authored by cebowleratibm, committed by ZarkoCA). Changed prior to commit: https://reviews.llvm.org/D88105?vs=293917=294222#toc

[clang] 64b8a63 - [NFC] [PPC] Add PowerPC expected IR tests for C99 complex

Author: Chris Bowler Date: 2020-09-24T23:28:40-04:00 New Revision: 64b8a633a872f25c8b3f9414c22165405400ea10 URL: https://github.com/llvm/llvm-project/commit/64b8a633a872f25c8b3f9414c22165405400ea10 DIFF: https://github.com/llvm/llvm-project/commit/64b8a633a872f25c8b3f9414c22165405400ea10.diff

[PATCH] D88275: [ASTMatchers] Add matcher `hasParentIgnoringImplicit`.

ymandel created this revision. ymandel added a reviewer: gribozavr2. Herald added a project: clang. ymandel requested review of this revision. This matcher combines the features of the `hasParent` matcher with skip-implicit-expression-nodes behavior of `ignoringImplicit`. It finds the first

[PATCH] D72218: [clang-tidy] new altera kernel name restriction check

Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/docs/clang-tidy/checks/altera-kernel-name-restriction.rst:4 +altera-kernel-name-restriction + + Please make same length as text above. CHANGES SINCE LAST ACTION

[PATCH] D72218: [clang-tidy] new altera kernel name restriction check

ffrankies updated this revision to Diff 294215. ffrankies edited the summary of this revision. ffrankies added a comment. Herald added a project: LLVM. Herald added a subscriber: llvm-commits. - Rebased code and fixed merge conflicts with D66564 - Added

[PATCH] D87953: [xray] Function coverage groups

This revision was automatically updated to reflect the committed changes. Closed by commit rG6f7fbdd2857f: [xray] Function coverage groups (authored by ianlevesque). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87953/new/

[clang] 6f7fbdd - [xray] Function coverage groups

Author: Ian Levesque Date: 2020-09-24T22:09:53-04:00 New Revision: 6f7fbdd2857fc8a7280afbb26fd4e1a6450069e4 URL: https://github.com/llvm/llvm-project/commit/6f7fbdd2857fc8a7280afbb26fd4e1a6450069e4 DIFF: https://github.com/llvm/llvm-project/commit/6f7fbdd2857fc8a7280afbb26fd4e1a6450069e4.diff

[clang] 8c98c88 - PR47176: Don't read from an inactive union member if a friend function

Author: Richard Smith Date: 2020-09-24T19:02:27-07:00 New Revision: 8c98c8803430804010da06a07cfb291dab59067c URL: https://github.com/llvm/llvm-project/commit/8c98c8803430804010da06a07cfb291dab59067c DIFF: https://github.com/llvm/llvm-project/commit/8c98c8803430804010da06a07cfb291dab59067c.diff

[PATCH] D54222: [clang-tidy] Add a check to detect returning static locals in public headers

jranieri-grammatech added a comment. In D54222#1290789 , @JonasToth wrote: > I agree with @Eugene.Zelenko that this check should rather live in > `bugprone-` as the issue its diagnosing is not LLVM specific. It is ok to add > an alias into the llvm

[PATCH] D87953: [xray] Function coverage groups

ianlevesque updated this revision to Diff 294213. ianlevesque added a comment. Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Add static function to the test case, update documentation. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D87953: [xray] Function coverage groups

dberris accepted this revision. dberris added a comment. This revision is now accepted and ready to land. Okay, I think I understand the motivation a little bit more now (i.e. sampling seems to be a useful thing to do anyway, regardless of whether we have the other facilities). Repository:

[PATCH] D88227: [clang-format] Add a SpaceBeforePointerQualifiers style option

mzeren-vmw added a comment. Perhaps PointerAlignment should be an enum of Left, Middle, Right? We maintain an internal patch that adds "ReferenceAlignment" as separate from PointerAlignment. So one could have: PointerAlignment: Middle ReferenceAlignment: Left Repository: rG LLVM Github

[PATCH] D87953: [xray] Function coverage groups

ianlevesque added a comment. I can update the docs @MaskRay, not a problem. I'll tweak the test a little too per your comment. @dberris I think if we wanted to control it in a non-'random' (the crc32 was chosen so it would be consistent even though it's not really predictable up front) way

[PATCH] D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata

lanza added inline comments. Comment at: clang/lib/CodeGen/CGObjC.cpp:466 + for (; begin != end; ++begin) +AppendFirstRuntimeProtocols(*begin, PDs); + rjmccall wrote: > Should this make an effort to avoid declaring redundant bases? e.g. > > ``` >

[PATCH] D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata

lanza marked 5 inline comments as done. lanza added a comment. Fixed and added a test under the `REDUNDANCY` prefix. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75574/new/ https://reviews.llvm.org/D75574

[PATCH] D87332: [profile] Add %t LLVM_PROFILE_FILE option to substitute $TMPDIR

ejvaughan accepted this revision. ejvaughan added a comment. This revision is now accepted and ready to land. Changes look good to me. Thanks for implementing this, Vedant! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87332/new/

[PATCH] D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata

lanza updated this revision to Diff 294207. lanza added a comment. Fix duplicate inheritance issue Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75574/new/ https://reviews.llvm.org/D75574 Files: clang/include/clang/AST/DeclObjC.h

[PATCH] D80344: [Windows SEH]: HARDWARE EXCEPTION HANDLING (MSVC -EHa) - Part 1

rjmccall added a comment. In D80344#2293954 , @tentzen wrote: > In D80344#2291926 , @rjmccall wrote: > >> Okay. I think we're on the same page about representation now. If you can >> come up with a good

[clang] 276f68e - Revert "Add a static_assert confirming that DiagnosticBuilder is small"

Author: Reid Kleckner Date: 2020-09-24T16:39:46-07:00 New Revision: 276f68eace7c27f6805d69a30a4b2f186e42c56c URL: https://github.com/llvm/llvm-project/commit/276f68eace7c27f6805d69a30a4b2f186e42c56c DIFF: https://github.com/llvm/llvm-project/commit/276f68eace7c27f6805d69a30a4b2f186e42c56c.diff

[clang] a32feed - Add a static_assert confirming that DiagnosticBuilder is small

Author: Reid Kleckner Date: 2020-09-24T16:38:41-07:00 New Revision: a32feed0dbeac7606d042d0d7e041c9eaf12cd51 URL: https://github.com/llvm/llvm-project/commit/a32feed0dbeac7606d042d0d7e041c9eaf12cd51 DIFF: https://github.com/llvm/llvm-project/commit/a32feed0dbeac7606d042d0d7e041c9eaf12cd51.diff

[PATCH] D68364: Implement C++20's P0784 (More constexpr containers)

leonardchan added a comment. Hi. Our GCC builders are failing with [182/12433] CXX kernel-x64-gcc/obj/kernel/lib/ktl/ktl.dummy-new.cc.o FAILED: kernel-x64-gcc/obj/kernel/lib/ktl/ktl.dummy-new.cc.o /b/s/w/ir/k/fuchsia/prebuilt/third_party/goma/linux-x64/gomacc

[clang] ecfc9b9 - [MS] For unknown ISAs, pass non-trivially copyable arguments indirectly

Author: Reid Kleckner Date: 2020-09-24T16:29:48-07:00 New Revision: ecfc9b971269a86b101cddf1fd9f0976be4096d0 URL: https://github.com/llvm/llvm-project/commit/ecfc9b971269a86b101cddf1fd9f0976be4096d0 DIFF: https://github.com/llvm/llvm-project/commit/ecfc9b971269a86b101cddf1fd9f0976be4096d0.diff

[clang] b8a50e9 - [MS] Simplify rules for passing C++ records

Author: Reid Kleckner Date: 2020-09-24T16:29:47-07:00 New Revision: b8a50e920704436ddcbe0cc9d2020935d7e37095 URL: https://github.com/llvm/llvm-project/commit/b8a50e920704436ddcbe0cc9d2020935d7e37095 DIFF: https://github.com/llvm/llvm-project/commit/b8a50e920704436ddcbe0cc9d2020935d7e37095.diff

[PATCH] D87953: [xray] Function coverage groups

dberris added a comment. There's two tensions I'm trying to resolve in my mind with this patch: - Is there a way to make this more deterministic, instead of using a random sampling mechanism based on the name? - This looks like it's a subset of a more general facility to group functions by

[PATCH] D80344: [Windows SEH]: HARDWARE EXCEPTION HANDLING (MSVC -EHa) - Part 1

tentzen added a comment. In D80344#2291926 , @rjmccall wrote: > Okay. I think we're on the same page about representation now. If you can > come up with a good replacement for "eha" in the intrinsic names, I think > this is pretty much ready to go.

[PATCH] D88227: [clang-format] Add a SpaceBeforePointerQualifiers style option

arichardson added inline comments. Comment at: clang/include/clang/Format/Format.h:2181 + /// \endcode + bool SpaceBeforePointerQualifiers; + MyDeveloperDay wrote: > I wonder if a request would ever say they wanted this Space before to be > Left/Middle/Right?

[PATCH] D88105: [NFC] [PPC] Add PowerPC expected IR tests for C99 complex

hubert.reinterpretcast added a comment. Off-list discussion seems to indicate that only the NFC portion of the patch was intended to be approved. That is, the scope of this patch is supposed to be https://reviews.llvm.org/D88105?id=293625 plus formatting changes. CHANGES SINCE LAST ACTION

[PATCH] D88239: [clang-format] Fix spaces around */& in multi-variable declarations

arichardson added inline comments. Comment at: clang/unittests/Format/FormatTest.cpp:6636 + verifyFormat("a* a = aaa, * b = bb,\n" + " * b = bbb, * d = ;", Style);

[PATCH] D88265: Fix comma with half vectors.

arames created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. arames requested review of this revision. The current half vector was enforcing an assert expecting "(LHS is half vector) == (RHS is half vector)" for comma. Repository: rG LLVM Github

[clang] c9b53b3 - Fix regex in test.

Author: Bill Wendling Date: 2020-09-24T15:21:28-07:00 New Revision: c9b53b3bf20d20d5752876be32a9e5887c702e53 URL: https://github.com/llvm/llvm-project/commit/c9b53b3bf20d20d5752876be32a9e5887c702e53 DIFF: https://github.com/llvm/llvm-project/commit/c9b53b3bf20d20d5752876be32a9e5887c702e53.diff

[PATCH] D87029: [AIX] Implement AIX special bitfield related alignment rules

Xiangling_L updated this revision to Diff 294157. Xiangling_L added a comment. Rebased on the NFC patch; CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87029/new/ https://reviews.llvm.org/D87029 Files: clang/lib/AST/RecordLayoutBuilder.cpp clang/lib/Sema/SemaDecl.cpp

[PATCH] D88263: Sema: remove unnecessary parameter for SwiftName handling (NFCI)

compnerd created this revision. compnerd added reviewers: aaron.ballman, gribozavr2. Herald added a project: clang. compnerd requested review of this revision. This code never actually did anything in the implementation. `mergeDeclAttribute` is declared as `static`, and referenced exactly once

[PATCH] D87808: [DebugInfo] Fix bug in constructor homing for classes with trivial constructors.

This revision was automatically updated to reflect the committed changes. Closed by commit rGc8df781e54a4: [DebugInfo] Fix bug in constructor homing with classes with trivial (authored by akhuang). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] c8df781 - [DebugInfo] Fix bug in constructor homing with classes with trivial

Author: Amy Huang Date: 2020-09-24T14:43:48-07:00 New Revision: c8df781e54a43593eafd993a5a5cd647866955f8 URL: https://github.com/llvm/llvm-project/commit/c8df781e54a43593eafd993a5a5cd647866955f8 DIFF: https://github.com/llvm/llvm-project/commit/c8df781e54a43593eafd993a5a5cd647866955f8.diff

[clang] f97b68e - Fix testcase.

Author: Bill Wendling Date: 2020-09-24T14:34:28-07:00 New Revision: f97b68ef4ddd28a685e502653768c2a34c314cba URL: https://github.com/llvm/llvm-project/commit/f97b68ef4ddd28a685e502653768c2a34c314cba DIFF: https://github.com/llvm/llvm-project/commit/f97b68ef4ddd28a685e502653768c2a34c314cba.diff

[PATCH] D88250: [CUDA] Added dim3/uint3 conversion functions to builtin vars.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG30514f0afa3e: [CUDA] Added conversion functions to builtin vars. (authored by tra). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[clang] 30514f0 - [CUDA] Added conversion functions to builtin vars.

Author: Artem Belevich Date: 2020-09-24T14:33:04-07:00 New Revision: 30514f0afa3ee1e6da6bf9c41e83c28e884f0740 URL: https://github.com/llvm/llvm-project/commit/30514f0afa3ee1e6da6bf9c41e83c28e884f0740 DIFF:

[PATCH] D88239: [clang-format] Fix spaces around */& in multi-variable declarations

MyDeveloperDay added inline comments. Comment at: clang/lib/Format/TokenAnnotator.cpp:2889 + return true; +return Style.PointerAlignment != FormatStyle::PAS_Left; } I wish more of these horrendous expressions all over the code were written like

[PATCH] D87720: Sema: add support for `__attribute__((__swift_private__))`

compnerd updated this revision to Diff 294167. compnerd marked an inline comment as done. compnerd added a comment. - address all feedback, add more test cases Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87720/new/

[PATCH] D87720: Sema: add support for `__attribute__((__swift_private__))`

compnerd marked an inline comment as done. compnerd added inline comments. Comment at: clang/lib/Sema/SemaDecl.cpp:2613-2614 SNA->getName(), AMK == Sema::AMK_Override); + else if (isa(Attr) && AMK == Sema::AMK_Override) +NewAttr =

[PATCH] D88227: [clang-format] Add a SpaceBeforePointerQualifiers style option

MyDeveloperDay added inline comments. Comment at: clang/include/clang/Format/Format.h:2181 + /// \endcode + bool SpaceBeforePointerQualifiers; + I wonder if a request would ever say they wanted this Space before to be Left/Middle/Right? perhaps this should

[PATCH] D75044: [AArch64] __builtin_return_address for PAuth.

This revision was automatically updated to reflect the committed changes. Closed by commit rG2a96f47c5ffc: [AArch64] __builtin_return_address for PAuth. (authored by danielkiss). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75044/new/

[PATCH] D52676: [clang-format] tweaked another case of lambda formatting

MyDeveloperDay added a comment. @oleg.smolsky I agree, what you have here covers a myriad of other cases and was committed in 2018, we can't call this commit a regression it was a feature ;-), if we want to improve the feature that is something else. Repository: rL LLVM CHANGES SINCE LAST

[PATCH] D87451: add new clang option -mno-xcoff-visibility

daltenty added inline comments. Comment at: llvm/include/llvm/Target/TargetMachine.h:265 + /// corresponding to -mno-xcoff-visibility. + bool getNoXCOFFVisibility() const { return Options.NoXCOFFVisibility; } + DiggerLin wrote: > jasonliu wrote: > > DiggerLin

[clang] 34ca5b3 - Remove stale assert.

Author: Bill Wendling Date: 2020-09-24T13:59:42-07:00 New Revision: 34ca5b3392ced08e2320fb4236cca5c7df4ec6e9 URL: https://github.com/llvm/llvm-project/commit/34ca5b3392ced08e2320fb4236cca5c7df4ec6e9 DIFF: https://github.com/llvm/llvm-project/commit/34ca5b3392ced08e2320fb4236cca5c7df4ec6e9.diff

[PATCH] D88195: Remove stale assert.

This revision was automatically updated to reflect the committed changes. Closed by commit rG34ca5b3392ce: Remove stale assert. (authored by void). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88195/new/ https://reviews.llvm.org/D88195 Files:

[PATCH] D88195: Remove stale assert.

nickdesaulniers accepted this revision. nickdesaulniers added a comment. In D88195#2293637 , @jyknight wrote: > In D88195#2293597 , @nickdesaulniers > wrote: > >> In D88195#2293589

[PATCH] D77229: [Analyzer] Avoid handling of LazyCompundVals in IteratorModeling

baloghadamsoftware updated this revision to Diff 294156. baloghadamsoftware added a comment. Test coverage was hugely increased by rebasing this patch to D88216 . Two kind of tests failed. This last update addreses them: 1. Handle the assignment operator of

[PATCH] D88195: Remove stale assert.

jyknight accepted this revision. jyknight added a comment. This revision is now accepted and ready to land. In D88195#2293597 , @nickdesaulniers wrote: > In D88195#2293589 , @void wrote: > >> Clarify commit

[PATCH] D88260: [NFC][FE] Replace TypeSize with StorageUnitSize

Xiangling_L created this revision. Xiangling_L added reviewers: jasonliu, hubert.reinterpretcast, efriedma. Herald added a project: clang. Herald added a subscriber: cfe-commits. Xiangling_L requested review of this revision. On some targets like AIX, last bitfield size is not always equal to

[PATCH] D88222: [AST] Use data-recursion when building ParentMap, avoid stack overflow.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG1ad94624f8a0: [AST] Use data-recursion when building ParentMap, avoid stack overflow. (authored by sammccall). Repository: rG LLVM Github

[clang] 1ad9462 - [AST] Use data-recursion when building ParentMap, avoid stack overflow.

Author: Sam McCall Date: 2020-09-24T22:49:44+02:00 New Revision: 1ad94624f8a092fbfcb74685e11243c186b04c8f URL: https://github.com/llvm/llvm-project/commit/1ad94624f8a092fbfcb74685e11243c186b04c8f DIFF: https://github.com/llvm/llvm-project/commit/1ad94624f8a092fbfcb74685e11243c186b04c8f.diff

[PATCH] D88222: [AST] Use data-recursion when building ParentMap, avoid stack overflow.

sammccall updated this revision to Diff 294154. sammccall marked an inline comment as done. sammccall added a comment. Address comments. Also remove a stale fixme, and simplify a little. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88222/new/

[PATCH] D52676: [clang-format] tweaked another case of lambda formatting

oleg.smolsky added a comment. @MyDeveloperDay that's the exact point. I authored this change to close a gap in some lambda formatting cases. The tests (existing, modified and added) express all relevant cases that I knew at the time. @jaafar, @curdeius - this is just C++ code. Nothing is set

[PATCH] D88195: Remove stale assert.

nickdesaulniers added a comment. In D88195#2293589 , @void wrote: > Clarify commit message. Phabricator unfortunately won't amend the review description when the commit message is updated; you'll need to manually edit the description via phab's UI for

[PATCH] D87451: add new clang option -mno-xcoff-visibility

DiggerLin marked an inline comment as done. DiggerLin added inline comments. Comment at: llvm/include/llvm/Target/TargetMachine.h:265 + /// corresponding to -mno-xcoff-visibility. + bool getNoXCOFFVisibility() const { return Options.NoXCOFFVisibility; } +

[PATCH] D88195: Remove stale assert.

void updated this revision to Diff 294150. void added a comment. Clarify commit message. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88195/new/ https://reviews.llvm.org/D88195 Files: clang/lib/AST/Stmt.cpp

[PATCH] D88195: Remove stale assert.

nickdesaulniers added a comment. In D88195#2293559 , @nickdesaulniers wrote: > In D88195#2293533 , @void wrote: > >> In D88195#2293165 , >> @nickdesaulniers wrote: >> >>>

[PATCH] D88195: Remove stale assert.

void added a comment. In D88195#2293559 , @nickdesaulniers wrote: > In D88195#2293533 , @void wrote: > >> In D88195#2293165 , >> @nickdesaulniers wrote: >> >>> I'm super

[PATCH] D87808: [DebugInfo] Fix bug in constructor homing where it would use ctor homing when a class only has copy/move constructors

akhuang updated this revision to Diff 294149. akhuang added a comment. Add to comment for lambdas. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87808/new/ https://reviews.llvm.org/D87808 Files: clang/lib/CodeGen/CGDebugInfo.cpp

[clang] 579c422 - [OPENMP]Fix PR47621: Variable used by task inside a template function is not made firstprivate by default

Author: Alexey Bataev Date: 2020-09-24T16:18:09-04:00 New Revision: 579c42225ac373688dbdbe3a1ce62986a6bf638a URL: https://github.com/llvm/llvm-project/commit/579c42225ac373688dbdbe3a1ce62986a6bf638a DIFF: https://github.com/llvm/llvm-project/commit/579c42225ac373688dbdbe3a1ce62986a6bf638a.diff

[PATCH] D74455: [MS] Pass aligned, non-trivially copyable things indirectly on x86

rnk abandoned this revision. rnk added a comment. I actually forgot about this change entirely and uploaded and landed a new one: https://reviews.llvm.org/D87923 I suppose they are functionally different: if we have an overaligned type that can be passed in registers due to [[trivial_abi]], we

[PATCH] D88195: Remove stale assert.

nickdesaulniers added a comment. In D88195#2293533 , @void wrote: > In D88195#2293165 , @nickdesaulniers > wrote: > >> I'm super confused between the commit message and initial hunk, that seem to >> make sense

[clang] f8a92ad - Remove dead branch identified by @rsmith on post-commit for D88236

Author: Erich Keane Date: 2020-09-24T13:05:15-07:00 New Revision: f8a92adfa242a94052f583ef7b483ae1b493ebdc URL: https://github.com/llvm/llvm-project/commit/f8a92adfa242a94052f583ef7b483ae1b493ebdc DIFF: https://github.com/llvm/llvm-project/commit/f8a92adfa242a94052f583ef7b483ae1b493ebdc.diff

[PATCH] D87349: [clang] adapt c++17 behavior for dependent decltype-specifiers

hokein added inline comments. Comment at: clang/include/clang/AST/Type.h:4499-4508 /// Internal representation of canonical, dependent /// decltype(expr) types. /// /// This class is used internally by the ASTContext to manage /// canonical, dependent types, only. Clients

[PATCH] D88195: Remove stale assert.

void added a comment. In D88195#2293165 , @nickdesaulniers wrote: > I'm super confused between the commit message and initial hunk, that seem to > make sense and probably should go in clang-11 if it's not too late, and the > additional tests for

[PATCH] D87349: [clang] adapt c++17 behavior for dependent decltype-specifiers

hokein updated this revision to Diff 294143. hokein marked 2 inline comments as done. hokein added a comment. store the actual expression in DecltypeTypeLoc and address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87349/new/

[PATCH] D88195: Remove stale assert.

void updated this revision to Diff 294141. void added a comment. Fix test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88195/new/ https://reviews.llvm.org/D88195 Files: clang/lib/AST/Stmt.cpp clang/test/Modules/Inputs/asm-goto/a.h

[PATCH] D87720: Sema: add support for `__attribute__((__swift_private__))`

compnerd updated this revision to Diff 294140. compnerd added a comment. add more tests to cover the confusing case of attribute merge handling Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87720/new/ https://reviews.llvm.org/D87720 Files:

[PATCH] D87029: [AIX] Implement AIX special bitfield related alignment rules

Xiangling_L marked an inline comment as done. Xiangling_L added a comment. In D87029#2285398 , @jasonliu wrote: > I think it would help the review if we could put the NFC portion(e.g. > TypeSize -> StorageUnitSize) to a new patch, and give some rationale

[clang] 9eba6b2 - Revert "[Modules] Add stats to measure performance of building and loading modules."

Author: Volodymyr Sapsai Date: 2020-09-24T12:36:06-07:00 New Revision: 9eba6b20a0579d3441e1b0f3cb3942f86d32679f URL: https://github.com/llvm/llvm-project/commit/9eba6b20a0579d3441e1b0f3cb3942f86d32679f DIFF:

[PATCH] D86895: [Modules] Add stats to measure performance of building and loading modules.

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

[PATCH] D88236: [PR47636] Fix tryEmitPrivate to handle non-constantarraytypes

erichkeane added a comment. Ok then, I'll take a look to see what it would take to make the MaterializeTemporaryExpr be created with the complete type here instead. Thanks! Comment at: clang/lib/CodeGen/CGExprConstant.cpp:2148 +CommonElementType == nullptr &&

[PATCH] D86895: [Modules] Add stats to measure performance of building and loading modules.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGc4bacc3c9b33: [Modules] Add stats to measure performance of building and loading modules. (authored by vsapsai). Repository: rG LLVM Github

[PATCH] D88236: [PR47636] Fix tryEmitPrivate to handle non-constantarraytypes

rsmith added inline comments. Comment at: clang/lib/CodeGen/CGExprConstant.cpp:2148 +CommonElementType == nullptr && !NumInitElts) { const ArrayType *AT = CGM.getContext().getAsArrayType(DestType); CommonElementType =

[clang] c4bacc3 - [Modules] Add stats to measure performance of building and loading modules.

Author: Volodymyr Sapsai Date: 2020-09-24T12:23:47-07:00 New Revision: c4bacc3c9b333bb7032fb96f41d6f5b851623132 URL: https://github.com/llvm/llvm-project/commit/c4bacc3c9b333bb7032fb96f41d6f5b851623132 DIFF:

[PATCH] D77229: [Analyzer] Avoid handling of LazyCompundVals in IteratorModeling

baloghadamsoftware added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/Iterator.cpp:330-336 +SVal getReturnIterator(const CallEvent ) { + Optional RetValUnderConstr = Call.getReturnValueUnderConstruction(); + if (RetValUnderConstr.hasValue()) +return

[PATCH] D88236: [PR47636] Fix tryEmitPrivate to handle non-constantarraytypes

rjmccall added a comment. Yeah, that would also be a completely reasonable approach. We already preserve the source spelling of the incomplete array type in the appropriate expression. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88236/new/ https://reviews.llvm.org/D88236

[PATCH] D88250: [CUDA] Added dim3/uint3 conversion functions to builtin vars.

tra marked an inline comment as done. tra added a comment. In D88250#2293346 , @tra wrote: > In D88250#2293270 , @jlebar wrote: > >> I know it comes in a separate change, but can we add a check to the >>

[PATCH] D88236: [PR47636] Fix tryEmitPrivate to handle non-constantarraytypes

rsmith added inline comments. Comment at: clang/test/CodeGenCXX/pr47636.cpp:2 +// RUN: %clang_cc1 -o - -emit-llvm -triple x86_64-linux-pc %s | FileCheck %s +int(&_rvref)[] {1,2,3,4}; +// CHECK: @_ZGR10intu_rvref_ = internal global [4 x i32] [i32 1, i32 2, i32 3, i32 4]

[PATCH] D88236: [PR47636] Fix tryEmitPrivate to handle non-constantarraytypes

erichkeane closed this revision. erichkeane added a comment. Gah, forgot 'Differential Revision' again :/ CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88236/new/ https://reviews.llvm.org/D88236 ___ cfe-commits mailing list

[PATCH] D70378: [LLD][COFF] Cover usage of LLD as a library

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGf2efb5742cc9: [LLD][COFF] Cover usage of LLD-as-a-library in tests (authored by aganea). Changed prior to commit:

[clang] 606a734 - [PR47636] Fix tryEmitPrivate to handle non-constantarraytypes

Author: Erich Keane Date: 2020-09-24T12:09:22-07:00 New Revision: 606a734755d1fb6c35a17680d0c251f834b79334 URL: https://github.com/llvm/llvm-project/commit/606a734755d1fb6c35a17680d0c251f834b79334 DIFF: https://github.com/llvm/llvm-project/commit/606a734755d1fb6c35a17680d0c251f834b79334.diff

[PATCH] D88236: [PR47636] Fix tryEmitPrivate to handle non-constantarraytypes

rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. Thanks, LGTM. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88236/new/ https://reviews.llvm.org/D88236 ___ cfe-commits mailing list

[PATCH] D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata

rjmccall added a comment. Okay, the result of internal review is that we're comfortable with this feature. Reviewers brought up the point that it would be interesting to add some way to ensure unique emission of a protocol, so that protocols that can't be non-runtime could avoid bloating

[PATCH] D88250: [CUDA] Added dim3/uint3 conversion functions to builtin vars.

tra marked an inline comment as done. tra added a comment. In D88250#2293270 , @jlebar wrote: > I know it comes in a separate change, but can we add a check to the > test-suite? Will do. Comment at:

[PATCH] D87451: add new clang option -mno-xcoff-visibility

jasonliu added inline comments. Comment at: llvm/include/llvm/Target/TargetMachine.h:265 + /// corresponding to -mno-xcoff-visibility. + bool getNoXCOFFVisibility() const { return Options.NoXCOFFVisibility; } + DiggerLin wrote: > daltenty wrote: > > This seems

[PATCH] D88250: [CUDA] Added dim3/uint3 conversion functions to builtin vars.

tra updated this revision to Diff 294125. tra edited the summary of this revision. tra added a comment. Fixed compatibility with pre-c++11 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88250/new/ https://reviews.llvm.org/D88250 Files:

[clang] d34c8c7 - Basic: add an extra newline for sphinx (NFC)

Author: Saleem Abdulrasool Date: 2020-09-24T18:51:10Z New Revision: d34c8c70aae2a5421337c2ccac91130c70511f94 URL: https://github.com/llvm/llvm-project/commit/d34c8c70aae2a5421337c2ccac91130c70511f94 DIFF: https://github.com/llvm/llvm-project/commit/d34c8c70aae2a5421337c2ccac91130c70511f94.diff

[PATCH] D87720: Sema: add support for `__attribute__((__swift_private__))`

compnerd updated this revision to Diff 294124. compnerd marked an inline comment as done. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87720/new/ https://reviews.llvm.org/D87720 Files: clang/include/clang/Basic/Attr.td

[PATCH] D87451: add new clang option -mno-xcoff-visibility

jasonliu added inline comments. Comment at: clang/docs/ClangCommandLineReference.rst:2310 +.. option:: -mno-xcoff-visibility + It's rare to see an option with only the negative form. Could we rename and make it a positive form somehow? Also we would need to

[PATCH] D87720: Sema: add support for `__attribute__((__swift_private__))`

compnerd marked 4 inline comments as done. compnerd added inline comments. Comment at: clang/include/clang/Basic/Attr.td:2189 + let Subjects = + SubjectList<[Enum, Function, Struct, TypedefName, + ObjCClassMethod, ObjCInstanceMethod, ObjCInterface,

[clang] cde7d90 - Revert "[OPENMP]Fix PR47621: Variable used by task inside a template function is not made firstprivate by default"

Author: Alexey Bataev Date: 2020-09-24T14:42:04-04:00 New Revision: cde7d90cc7c20d73d14225517cf11ffc6073018a URL: https://github.com/llvm/llvm-project/commit/cde7d90cc7c20d73d14225517cf11ffc6073018a DIFF: https://github.com/llvm/llvm-project/commit/cde7d90cc7c20d73d14225517cf11ffc6073018a.diff

[PATCH] D88236: [PR47636] Fix tryEmitPrivate to handle non-constantarraytypes

erichkeane updated this revision to Diff 294121. erichkeane marked an inline comment as done. erichkeane added a comment. @rjmccall Done! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88236/new/ https://reviews.llvm.org/D88236 Files: clang/lib/CodeGen/CGExprConstant.cpp

  1   2   3   >