[PATCH] D92936: [Sema] Fix deleted function problem in implicitly movable test

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

[PATCH] D93683: [clangd] Do not take stale definition from the static index.

sammccall added a comment. In D93683#2469553 , @ArcsinX wrote: > In D93683#2468842 , @sammccall wrote: > >> I'm not 100% sure this bug would block adding the equivalent change for >> `fuzzyFind` though - it'll affe

[PATCH] D93683: [clangd] Do not take stale definition from the static index.

ArcsinX updated this revision to Diff 313504. ArcsinX added a comment. Add comment. Fix possible `MergeIndexTest.LookupRemovedDefinition` test failure. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93683/new/ https://reviews.llvm.org/D93683 Files:

[PATCH] D93683: [clangd] Do not take stale definition from the static index.

ArcsinX marked an inline comment as done. ArcsinX added a comment. In D93683#2469584 , @sammccall wrote: > In D93683#2469553 , @ArcsinX wrote: > >> In D93683#2468842 , @sammc

[PATCH] D86844: [LoopDeletion] Allows deletion of possibly infinite side-effect free loops

fhahn added inline comments. Comment at: llvm/test/Other/loop-deletion-printer.ll:17 -define void @deleteme() { +define void @deleteme() willreturn { entry: atmnpatel wrote: > fhahn wrote: > > Is this change related to the patch? Same for the other test change

[clang] 25a02c3 - Revert "PR24076, PR33655, C++ CWG 1558: Consider the instantiation-dependence of"

Author: Adrian Kuegel Date: 2020-12-23T12:31:52+01:00 New Revision: 25a02c3d1a688d3cd18faef96c75fa553efbbac7 URL: https://github.com/llvm/llvm-project/commit/25a02c3d1a688d3cd18faef96c75fa553efbbac7 DIFF: https://github.com/llvm/llvm-project/commit/25a02c3d1a688d3cd18faef96c75fa553efbbac7.diff

[PATCH] D93733: [NFC] replace resize calls with resize_for_overwrite

njames93 created this revision. Herald added subscribers: dexonsmith, asbirlea, hiraditya. njames93 published this revision for review. Herald added projects: clang, LLDB, LLVM. Herald added subscribers: llvm-commits, lldb-commits, cfe-commits. Also replace a few calls to resize followed by zeroin

[PATCH] D93758: [format][NFC] Use unsigned char as the base of all enums in FormatStyle

njames93 created this revision. njames93 added a reviewer: MyDeveloperDay. njames93 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This removes alot of unnecessary padding, trimming the size of the struct from 728->608 on 64 bit platforms

[PATCH] D93600: [clangd] When querying drivers by binary, look in PATH too

rapgenic updated this revision to Diff 313528. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93600/new/ https://reviews.llvm.org/D93600 Files: clang-tools-extra/clangd/QueryDriverDatabase.cpp clang-tools-extra/clangd/test/system-include-extractor.test Index: clang-tools-extra/clang

[PATCH] D93600: [clangd] When querying drivers by binary, look in PATH too

rapgenic marked 3 inline comments as done. rapgenic added a comment. Done! My email is: Giulio Girardi Thank you for your help! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93600/new/ https://reviews.llvm.org/D93600 ___ cfe-commits mailin

[clang] eb9483b - [format] Add overload to parseConfiguration that accept llvm::MemoryBufferRef

Author: Nathan James Date: 2020-12-23T12:08:29Z New Revision: eb9483b21053656b885f13ccfe41bfa76eb3df45 URL: https://github.com/llvm/llvm-project/commit/eb9483b21053656b885f13ccfe41bfa76eb3df45 DIFF: https://github.com/llvm/llvm-project/commit/eb9483b21053656b885f13ccfe41bfa76eb3df45.diff LOG:

[PATCH] D93633: [format] Add overload to parseConfiguration that accept llvm::MemoryBufferRef

This revision was automatically updated to reflect the committed changes. Closed by commit rGeb9483b21053: [format] Add overload to parseConfiguration that accept llvm::MemoryBufferRef (authored by njames93). Changed prior to commit: https://reviews.llvm.org/D93633?vs=313096&id=313530#toc Repo

[PATCH] D93401: [flang][driver] Add support for `-D`, `-U`

FarisRehman updated this revision to Diff 313532. FarisRehman marked 5 inline comments as done. FarisRehman added a comment. Address review comments This revision addresses comments by @awarzynski Summary of changes: - Update commit summary - Use Doxygen style for input/output parameters - Make

[PATCH] D91000: [clang-tidy] CERT MSC24-C Obsolescent Functions check

steakhal requested changes to this revision. steakhal added a comment. This revision now requires changes to proceed. In D91000#2465514 , @ktomi996 wrote: > In D91000#2382562 , @steakhal wrote: > >> Quoting the revi

[PATCH] D91000: [clang-tidy] CERT MSC24-C Obsolescent Functions check

lebedev.ri added a comment. In D91000#2469880 , @steakhal wrote: > In D91000#2465514 , @ktomi996 wrote: > >> In D91000#2382562 , @steakhal wrote: >> >>> Quoting the revision

[PATCH] D93224: [RFC][analyzer] Use the MacroExpansionContext for macro expansions in plists

steakhal planned changes to this revision. steakhal added inline comments. Comment at: clang/lib/StaticAnalyzer/Core/PlistDiagnostics.cpp:20-21 #include "clang/Basic/Version.h" #include "clang/CrossTU/CrossTranslationUnit.h" #include "clang/Frontend/ASTUnit.h" #include "clang

[PATCH] D91000: [clang-tidy] CERT MSC24-C Obsolescent Functions check

steakhal added a comment. In D91000#2469884 , @lebedev.ri wrote: > In D91000#2469880 , @steakhal wrote: > >> In D91000#2465514 , @ktomi996 wrote: >> >>> In D91000#2382562

[PATCH] D93453: [flang][driver] Add support for `-I`

FarisRehman updated this revision to Diff 313536. FarisRehman added a comment. Clean up test Summary of changes: - Clean up test include_header.f90 - Rename SearchDirectoriesFromDashI to searchDirectoriesFromDashI Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://revie

[PATCH] D91000: [clang-tidy] CERT MSC24-C Obsolescent Functions check

lebedev.ri added a comment. In D91000#2469894 , @steakhal wrote: > In D91000#2469884 , @lebedev.ri > wrote: > >> In D91000#2469880 , @steakhal wrote: >> >>> In D91000#246551

[PATCH] D92039: [-Wcalled-once-parameter] Introduce 'called_once' attribute

vsavchenko added inline comments. Comment at: clang/include/clang/Analysis/Analyses/CalledOnceCheck.h:32 +/// \enum SwitchSkipped -- there is no call if none of the cases appies. +/// \enum LoopEntered -- no call when the loop is entered. +/// \enum LoopSkipped -- no call when th

[PATCH] D91000: [clang-tidy] CERT MSC24-C Obsolescent Functions check

steakhal added a comment. In D91000#2469898 , @lebedev.ri wrote: > I think the question is, *why* are these checks being implemented? > Just to claim that for some particular rule there is a check, and cross it > off a list? Initially, yes. I think one c

[PATCH] D93758: [format][NFC] Use unsigned char as the base of all enums in FormatStyle

MyDeveloperDay added a comment. We should probably check the docs/tools/dump_format_style.py still works Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93758/new/ https://reviews.llvm.org/D93758 ___ cfe-c

[PATCH] D91927: [X86] Add x86_amx type for intel AMX.

pengfei added a comment. > In my test case, it is transformed after Combine redundant instructions. Can we disable it for AMX type? The pointer to AMX type is meaningless and may result in bad perfomance. Comment at: llvm/lib/Target/X86/X86LowerAMXType.cpp:300 +// If

[PATCH] D92039: [-Wcalled-once-parameter] Introduce 'called_once' attribute

vsavchenko added inline comments. Comment at: clang/lib/Analysis/CalledOnceCheck.cpp:163 + } + static bool canBeCalled(Kind K) { +return (K & DefinitelyCalled) == DefinitelyCalled && K != Reported; NoQ wrote: > "Can be called" sounds like "Someone's allowed

[PATCH] D93758: [format][NFC] Use unsigned char as the base of all enums in FormatStyle

njames93 added a comment. In D93758#2469967 , @MyDeveloperDay wrote: > We should probably check the docs/tools/dump_format_style.py still works Good catch, I'll update the script to optionally take an underlying type argument. Should that be part of thi

[clang] 5426b2f - [clang-format] PR48535 clang-format Incorrectly Removes Space After C Style Cast When Type Is Not a Pointer

Author: mydeveloperday Date: 2020-12-23T14:45:14Z New Revision: 5426b2f9ed9f6f3a3e1d6452325f7a49a5d08ec4 URL: https://github.com/llvm/llvm-project/commit/5426b2f9ed9f6f3a3e1d6452325f7a49a5d08ec4 DIFF: https://github.com/llvm/llvm-project/commit/5426b2f9ed9f6f3a3e1d6452325f7a49a5d08ec4.diff LOG

[clang] 031743c - [clang-format] PR48539 ReflowComments breaks Qt translation comments

Author: mydeveloperday Date: 2020-12-23T14:45:14Z New Revision: 031743cb5b3c6c2df85a67d8533ef72a95e76cdc URL: https://github.com/llvm/llvm-project/commit/031743cb5b3c6c2df85a67d8533ef72a95e76cdc DIFF: https://github.com/llvm/llvm-project/commit/031743cb5b3c6c2df85a67d8533ef72a95e76cdc.diff LOG

[PATCH] D93626: [clang-format] PR48535 clang-format Incorrectly Removes Space After C Style Cast When Type Is Not a Pointer

This revision was automatically updated to reflect the committed changes. Closed by commit rG5426b2f9ed9f: [clang-format] PR48535 clang-format Incorrectly Removes Space After C Style… (authored by MyDeveloperDay). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.

[PATCH] D93490: [clang-format] PR48539 ReflowComments breaks Qt translation comments

This revision was automatically updated to reflect the committed changes. Closed by commit rG031743cb5b3c: [clang-format] PR48539 ReflowComments breaks Qt translation comments (authored by MyDeveloperDay). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org

[PATCH] D93630: [Attr] Apply GNU-style attributes to expression statements

vsavchenko updated this revision to Diff 313550. vsavchenko added a comment. Refine condition for statement attributes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93630/new/ https://reviews.llvm.org/D93630 Files: clang/lib/Parse/ParseStmt.cpp

[PATCH] D93763: [clangd] Add a flag to disable the documentLinks LSP request

rapgenic created this revision. Herald added subscribers: usaxena95, kadircet, arphaman. rapgenic requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang. On some editors (namely VSCode) documentLinks are rendered with an unde

[PATCH] D93630: [Attr] Apply GNU-style attributes to expression statements

vsavchenko added a comment. In D93630#2468853 , @aaron.ballman wrote: > Yeah, I kind of figured that might be the cause. I'm not 100% convinced (one > way or the other) if the suppress attribute should get a GNU spelling. The > `[[]]` spellings are avai

[clang-tools-extra] 2522fa0 - [clangd] Do not take stale definition from the static index.

Author: Aleksandr Platonov Date: 2020-12-23T18:21:38+03:00 New Revision: 2522fa053b62520ae48b4b27117ca003a2c878ab URL: https://github.com/llvm/llvm-project/commit/2522fa053b62520ae48b4b27117ca003a2c878ab DIFF: https://github.com/llvm/llvm-project/commit/2522fa053b62520ae48b4b27117ca003a2c878ab.

[PATCH] D93683: [clangd] Do not take stale definition from the static index.

This revision was automatically updated to reflect the committed changes. Closed by commit rG2522fa053b62: [clangd] Do not take stale definition from the static index. (authored by ArcsinX). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93683/new/ h

[PATCH] D83892: [clang][cli] Port CodeGen option flags to new option parsing system

thakis added a comment. Heads-up: Looks like one of these changes, likely this one for CodeGen, made sanitizer output way larger: https://bugs.chromium.org/p/chromium/issues/detail?id=1161230 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83892/new

[PATCH] D93758: [format][NFC] Use unsigned char as the base of all enums in FormatStyle

njames93 updated this revision to Diff 313559. njames93 added a comment. - Update dump_format_style script Only tweaked regex to support enum : ... { Decided there wasn't much to gain from supporting nested types etc Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://r

[PATCH] D93758: [format][NFC] Use unsigned char as the base of all enums in FormatStyle

MyDeveloperDay added a comment. I don't have any major issues with this other than it makes Format.h a bit more ugly. (there are more ugly and harder to understand pieces of code than this change!) We'll only normally really have 1 of these in play at any one time, but it is passed around quit

[PATCH] D93758: [format][NFC] Use unsigned char as the base of all enums in FormatStyle

njames93 added a comment. In D93758#2470124 , @MyDeveloperDay wrote: > I don't have any major issues with this other than it makes Format.h a bit > more ugly. (there are more ugly and harder to understand pieces of code than > this change!) > > We'll on

[PATCH] D93747: Rename debug linkage name with -funique-internal-linkage-names

tmsriram added a comment. In D93747#2469556 , @hoy wrote: >> In D93656 , @dblaikie wrote: >> Though the C case is interesting - it means you'll end up with C functions >> with the same DWARF 'name' but different linkage na

[PATCH] D93656: Moving UniqueInternalLinkageNamesPass to the start of IR pipelines.

hoy updated this revision to Diff 313566. hoy edited the summary of this revision. hoy added a comment. Checking pipeline in clang test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93656/new/ https://reviews.llvm.org/D93656 Files: clang/lib/Co

[PATCH] D92024: [clang] Implement P0692R1 from C++20 (access checking on specializations and instantiations)

aorlov added a comment. Please, somebody look at this patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92024/new/ https://reviews.llvm.org/D92024 ___ cfe-commits mailing list cfe-commits@lists.llvm.o

[PATCH] D83892: [clang][cli] Port CodeGen option flags to new option parsing system

thakis added a comment. I verified it's due to this change. https://bugs.chromium.org/p/chromium/issues/detail?id=1161230#c15 has a stand-alone repro. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83892/new/ https://reviews.llvm.org/D83892 _

[PATCH] D93747: Rename debug linkage name with -funique-internal-linkage-names

hoy added a comment. In D93747#2470178 , @tmsriram wrote: > In D93747#2469556 , @hoy wrote: > >>> In D93656 , @dblaikie wrote: >>> Though the C case is interesting - it means you'll

[PATCH] D93747: Rename debug linkage name with -funique-internal-linkage-names

tmsriram added a comment. In D93747#2470189 , @hoy wrote: > In D93747#2470178 , @tmsriram wrote: > >> In D93747#2469556 , @hoy wrote: >> In D93656

[PATCH] D93769: RFC [clang] Add support for option -ffp-eval-method and extend #pragma float_control similarly

mibintc created this revision. mibintc added reviewers: kpn, sepavloff, andrew.w.kaylor, rjmccall. Herald added subscribers: dexonsmith, dang. mibintc requested review of this revision. Herald added a project: clang. The Intel compiler supports the option -fp-model={source|double|extended} which

[PATCH] D83892: [clang][cli] Port CodeGen option flags to new option parsing system

thakis added a comment. I reverted this (and a bunch of stuff that landed on top of it) in 7ad666798f12456d9e663e763e17e29007c3728d for now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.

[PATCH] D93769: RFC [clang] Add support for option -ffp-eval-method and extend #pragma float_control similarly

mibintc added inline comments. Comment at: clang/lib/Sema/SemaAttr.cpp:429 + case PFC_Source: +//Builder.defineMacro("__FLT_EVAL_METHOD__", Twine("0")); +NewFPFeatures.setFPEvalMethodOverride(LangOptions::FEM_Source); @rjmccall I would like to push a new

[PATCH] D92024: [clang] Implement P0692R1 from C++20 (access checking on specializations and instantiations)

Quuxplusone added inline comments. Comment at: clang/lib/Parse/ParseTemplate.cpp:265 + // We don't use `SuppressAccessChecks` here because + // it supresses ALL the dignostics kinds, but we need to keep some of them. + // For instance marked as unavailable: Sp

[clang] b920adf - This is a test commit

Author: Alan Phipps Date: 2020-12-23T12:57:27-06:00 New Revision: b920adf3b4f16bef8dde937b67874d8e8ac1030e URL: https://github.com/llvm/llvm-project/commit/b920adf3b4f16bef8dde937b67874d8e8ac1030e DIFF: https://github.com/llvm/llvm-project/commit/b920adf3b4f16bef8dde937b67874d8e8ac1030e.diff L

[clang] bbd758a - Revert "This is a test commit"

Author: Alan Phipps Date: 2020-12-23T13:04:37-06:00 New Revision: bbd758a7913b1c374ca26e5a734a01200754fe0e URL: https://github.com/llvm/llvm-project/commit/bbd758a7913b1c374ca26e5a734a01200754fe0e DIFF: https://github.com/llvm/llvm-project/commit/bbd758a7913b1c374ca26e5a734a01200754fe0e.diff L

[clang] 1876a29 - Revert more changes that landed on top of 741978d727

Author: Nico Weber Date: 2020-12-23T14:20:21-05:00 New Revision: 1876a2914fe0bedf50f7be6a305f5bf35493e496 URL: https://github.com/llvm/llvm-project/commit/1876a2914fe0bedf50f7be6a305f5bf35493e496 DIFF: https://github.com/llvm/llvm-project/commit/1876a2914fe0bedf50f7be6a305f5bf35493e496.diff LO

[PATCH] D83892: [clang][cli] Port CodeGen option flags to new option parsing system

thakis added a comment. ...and two more in 1876a2914fe0bedf50f7be6a305f5bf35493e496 . Sorry for the churn! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83892/new/ https://revie

[clang] 4c37453 - clang: Build and run FrontendTests with CLANG_ENABLE_STATIC_ANALYZER=OFF too

Author: Nico Weber Date: 2020-12-23T14:27:09-05:00 New Revision: 4c37453a04f942dd676af1eda5d0760d4ffe8927 URL: https://github.com/llvm/llvm-project/commit/4c37453a04f942dd676af1eda5d0760d4ffe8927 DIFF: https://github.com/llvm/llvm-project/commit/4c37453a04f942dd676af1eda5d0760d4ffe8927.diff LO

[clang] 34e70d7 - Append ".__part." to every basic block section symbol.

Author: Sriraman Tallam Date: 2020-12-23T11:35:44-08:00 New Revision: 34e70d722dfd0e73d460802e8d43d3a885d24784 URL: https://github.com/llvm/llvm-project/commit/34e70d722dfd0e73d460802e8d43d3a885d24784 DIFF: https://github.com/llvm/llvm-project/commit/34e70d722dfd0e73d460802e8d43d3a885d24784.dif

[PATCH] D93082: Append ".__part." to all basic block section symbols.

This revision was automatically updated to reflect the committed changes. Closed by commit rG34e70d722dfd: Append ".__part." to every basic block section symbol. (authored by tmsriram). Herald added a project: clang. Herald added a subscriber: cfe-commits. Changed prior to commit: https://revie

[PATCH] D93747: Rename debug linkage name with -funique-internal-linkage-names

dblaikie added a comment. Please remove the clang test change - if this is an LLVM change with LLVM test coverage, that's enough. (we don't generally test every LLVM change from both LLVM and Clang) I'd still be curious if you could look around to see whether there are other cases of function

[PATCH] D87188: [InstCombine] Canonicalize SPF to abs intrinc

thakis added a comment. Heads up: Breaks a test for us: https://bugs.chromium.org/p/chromium/issues/detail?id=1161542 (No reduced repro yet, might be UB, just fyi at this point.) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87188/new/ https://re

[PATCH] D93656: Moving UniqueInternalLinkageNamesPass to the start of IR pipelines.

dblaikie added inline comments. Comment at: clang/test/CodeGen/unique-internal-linkage-names.cpp:48-49 +// LPIPELINE: Unique Internal Linkage Names +// NPIPELINE: Running pass: UniqueInternalLinkageNamesPass // PLAIN: @_ZL4glob = internal global Does this test

[PATCH] D87188: [InstCombine] Canonicalize SPF to abs intrinc

lebedev.ri added a comment. In D87188#2470392 , @thakis wrote: > Heads up: Breaks a test for us: > https://bugs.chromium.org/p/chromium/issues/detail?id=1161542 > > (No reduced repro yet, might be UB, just fyi at this point.) Thanks for headsup. For now

[PATCH] D93656: Moving UniqueInternalLinkageNamesPass to the start of IR pipelines.

hoy added inline comments. Comment at: clang/test/CodeGen/unique-internal-linkage-names.cpp:48-49 +// LPIPELINE: Unique Internal Linkage Names +// NPIPELINE: Running pass: UniqueInternalLinkageNamesPass // PLAIN: @_ZL4glob = internal global dblaikie wrote: > D

[PATCH] D86844: [LoopDeletion] Allows deletion of possibly infinite side-effect free loops

atmnpatel updated this revision to Diff 313593. atmnpatel added a comment. As expected, @fhahn was right, adding willreturn to all those tests was an artifact from previous revisions of this patch and it passed the tests so I didn't pay them any mind, but I removed them now. Repository: rG L

[PATCH] D93758: [format][NFC] Use unsigned char as the base of all enums in FormatStyle

This revision was automatically updated to reflect the committed changes. Closed by commit rGc7e825b910a9: [format][NFC] Use unsigned char as the base of all enums in FormatStyle (authored by njames93). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D9

[clang] c7e825b - [format][NFC] Use unsigned char as the base of all enums in FormatStyle

Author: Nathan James Date: 2020-12-23T20:27:44Z New Revision: c7e825b910a96c42bda1de4e7fb34c369da76625 URL: https://github.com/llvm/llvm-project/commit/c7e825b910a96c42bda1de4e7fb34c369da76625 DIFF: https://github.com/llvm/llvm-project/commit/c7e825b910a96c42bda1de4e7fb34c369da76625.diff LOG:

[PATCH] D93772: [Clang][Driver] Fix read-after-free when using /clang:

aganea created this revision. aganea added reviewers: thakis, hans, rnk, neerajksingh. aganea requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. As described in https://bugs.llvm.org/show_bug.cgi?id=42501 Fixes PR42501 Repository: rG LLVM

[PATCH] D93775: [WebAssembly] Prototype extending pairwise add instructions

tlively created this revision. tlively added a reviewer: aheejin. Herald added subscribers: wingo, ecnelises, sunfish, hiraditya, jgravelle-google, sbc100, dschuff. tlively requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits.

[PATCH] D93656: Moving UniqueInternalLinkageNamesPass to the start of IR pipelines.

dblaikie added inline comments. Comment at: clang/test/CodeGen/unique-internal-linkage-names.cpp:48-49 +// LPIPELINE: Unique Internal Linkage Names +// NPIPELINE: Running pass: UniqueInternalLinkageNamesPass // PLAIN: @_ZL4glob = internal global hoy wrote: > d

[PATCH] D87188: [InstCombine] Canonicalize SPF to abs intrinc

lebedev.ri added a comment. In D87188#2467915 , @nlopes wrote: > This patch regressed the following test of Transforms/InstCombine/abs-1.ll: > (need to drop NSW in this case). > > define i8 @nabs_canonical_3(i8 %x) { > %0: > %cmp = icmp slt i8 %x,

[clang] 374f1d8 - [clang-format] Fix handling of TextProto comments

Author: Björn Schäpers Date: 2020-12-23T22:07:13+01:00 New Revision: 374f1d81febf8143e8e633296a42c2311699c5b3 URL: https://github.com/llvm/llvm-project/commit/374f1d81febf8143e8e633296a42c2311699c5b3 DIFF: https://github.com/llvm/llvm-project/commit/374f1d81febf8143e8e633296a42c2311699c5b3.diff

[PATCH] D93163: [clang-format] Fix handling of ## comments in TextProto

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG374f1d81febf: [clang-format] Fix handling of TextProto comments (authored by HazardyKnusperkeks). Repository: rG LLVM Github Monorepo CHANGES SIN

[clang] 47877c9 - [clang-format] Add SpaceBeforeCaseColon option

Author: Björn Schäpers Date: 2020-12-23T22:07:14+01:00 New Revision: 47877c9079c27f19a954b660201ea47717c82fec URL: https://github.com/llvm/llvm-project/commit/47877c9079c27f19a954b660201ea47717c82fec DIFF: https://github.com/llvm/llvm-project/commit/47877c9079c27f19a954b660201ea47717c82fec.diff

[PATCH] D93240: [clang-format] Add SpaceBeforeCaseColon option

This revision was automatically updated to reflect the committed changes. Closed by commit rG47877c9079c2: [clang-format] Add SpaceBeforeCaseColon option (authored by HazardyKnusperkeks). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93240/new/ http

[PATCH] D93776: [clang-format] Add StatementAttributeLikeMacros option

HazardyKnusperkeks created this revision. HazardyKnusperkeks added reviewers: MyDeveloperDay, curdeius, JVApen. HazardyKnusperkeks added a project: clang-format. HazardyKnusperkeks requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This allows

[PATCH] D93776: [clang-format] Add StatementAttributeLikeMacros option

HazardyKnusperkeks added inline comments. Comment at: clang/lib/Format/FormatToken.h:54 TYPE(FunctionTypeLParen) \ + TYPE(StatementAttributeLikeMacro) \ TYPE(ImplicitStringLite

[PATCH] D93747: Rename debug linkage name with -funique-internal-linkage-names

hoy added a comment. In D93747#2470387 , @dblaikie wrote: > Please remove the clang test change - if this is an LLVM change with LLVM > test coverage, that's enough. (we don't generally test every LLVM change from > both LLVM and Clang) Sounds good. >

[PATCH] D93747: Rename debug linkage name with -funique-internal-linkage-names

hoy updated this revision to Diff 313607. hoy added a comment. Undoing changes to the clang test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93747/new/ https://reviews.llvm.org/D93747 Files: llvm/include/llvm/IR/DebugInfoMetadata.h llvm/lib

[PATCH] D93656: Moving UniqueInternalLinkageNamesPass to the start of IR pipelines.

aeubanks added inline comments. Comment at: clang/test/CodeGen/unique-internal-linkage-names.cpp:48-49 +// LPIPELINE: Unique Internal Linkage Names +// NPIPELINE: Running pass: UniqueInternalLinkageNamesPass // PLAIN: @_ZL4glob = internal global dblaikie wrote

[PATCH] D93776: [clang-format] Add StatementAttributeLikeMacros option

curdeius added a comment. Seems okay at the first glance. I'm wondering if we can find a better name though. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93776/new/ https://reviews.llvm.org/D93776 ___

[PATCH] D93656: Moving UniqueInternalLinkageNamesPass to the start of IR pipelines.

hoy added inline comments. Comment at: clang/test/CodeGen/unique-internal-linkage-names.cpp:48-49 +// LPIPELINE: Unique Internal Linkage Names +// NPIPELINE: Running pass: UniqueInternalLinkageNamesPass // PLAIN: @_ZL4glob = internal global aeubanks wrote: > d

[PATCH] D93656: Moving UniqueInternalLinkageNamesPass to the start of IR pipelines.

aeubanks added inline comments. Comment at: clang/test/CodeGen/unique-internal-linkage-names.cpp:48-49 +// LPIPELINE: Unique Internal Linkage Names +// NPIPELINE: Running pass: UniqueInternalLinkageNamesPass // PLAIN: @_ZL4glob = internal global hoy wrote: > a

[PATCH] D93776: [clang-format] Add StatementAttributeLikeMacros option

HazardyKnusperkeks added a comment. In D93776#2470524 , @curdeius wrote: > Seems okay at the first glance. I'm wondering if we can find a better name > though. I'm all in for a better name. :) I could not think of any (my first shot was `IgnoredMacros`,

[clang-tools-extra] 74b3ace - [clangd] Fix case mismatch crash on in CDB on windows after 92dd077af1ff8

Author: Sam McCall Date: 2020-12-23T22:42:45+01:00 New Revision: 74b3acefc7b6355e89bb9b09dc88a5948f65c342 URL: https://github.com/llvm/llvm-project/commit/74b3acefc7b6355e89bb9b09dc88a5948f65c342 DIFF: https://github.com/llvm/llvm-project/commit/74b3acefc7b6355e89bb9b09dc88a5948f65c342.diff LO

[PATCH] D93656: Moving UniqueInternalLinkageNamesPass to the start of IR pipelines.

hoy added inline comments. Comment at: clang/test/CodeGen/unique-internal-linkage-names.cpp:48-49 +// LPIPELINE: Unique Internal Linkage Names +// NPIPELINE: Running pass: UniqueInternalLinkageNamesPass // PLAIN: @_ZL4glob = internal global aeubanks wrote: > h

[PATCH] D92195: [OPENMP50]Mapping of the subcomponents with the 'default' mappers.

abhinavgaba added a comment. The patch currently asserts for the following: #include typedef struct { int a; double *b; } C; #pragma omp declare mapper(default: C s) map(s.a, s.b[0:2]) int

[PATCH] D93273: [CodeGen][ObjC] Destroy callee-destroyed arguments in the caller function when the receiver is nil

rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. Functionally LGTM. Minor suggestion. Comment at: clang/lib/CodeGen/CGObjCMac.cpp:2169 + return false; +} + Is this something that could reasonably just

[PATCH] D93763: [clangd] Add a flag to disable the documentLinks LSP request

njames93 added a comment. Can you apply the clang-format patch suggested by the pre-merge bot. Alternatively, if you have a recent version of clang-format and the appropriate git-hooks you should be able to run `git clang-format` for the same effect. Repository: rG LLVM Github Monorepo CHAN

[clang] 245218b - Basic: Support named pipes natively in SourceManager and FileManager

Author: Duncan P. N. Exon Smith Date: 2020-12-23T14:57:41-08:00 New Revision: 245218bb355599771ba43a0fe1449d1670f2666c URL: https://github.com/llvm/llvm-project/commit/245218bb355599771ba43a0fe1449d1670f2666c DIFF: https://github.com/llvm/llvm-project/commit/245218bb355599771ba43a0fe1449d1670f2

[PATCH] D92531: Basic: Support named pipes natively in SourceManager

This revision was automatically updated to reflect the committed changes. Closed by commit rG245218bb3555: Basic: Support named pipes natively in SourceManager and FileManager (authored by dexonsmith). Changed prior to commit: https://reviews.llvm.org/D92531?vs=311004&id=313618#toc Repository:

[clang] 3ee43ad - Basic: Add native support for stdin to SourceManager and FileManager

Author: Duncan P. N. Exon Smith Date: 2020-12-23T15:18:50-08:00 New Revision: 3ee43adfb20d5dc56b7043b314bd22f457c55483 URL: https://github.com/llvm/llvm-project/commit/3ee43adfb20d5dc56b7043b314bd22f457c55483 DIFF: https://github.com/llvm/llvm-project/commit/3ee43adfb20d5dc56b7043b314bd22f457c5

[PATCH] D93148: Basic: Add native support for stdin to SourceManager and FileManager

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG3ee43adfb20d: Basic: Add native support for stdin to SourceManager and FileManager (authored by dexonsmith). Changed prior to commit: https://revi

[PATCH] D93656: Moving UniqueInternalLinkageNamesPass to the start of IR pipelines.

dblaikie added inline comments. Comment at: clang/test/CodeGen/unique-internal-linkage-names.cpp:48-49 +// LPIPELINE: Unique Internal Linkage Names +// NPIPELINE: Running pass: UniqueInternalLinkageNamesPass // PLAIN: @_ZL4glob = internal global hoy wrote: > a

[PATCH] D90507: [Driver] Add DWARF64 flag: -gdwarf64

ayermolo updated this revision to Diff 313621. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90507/new/ https://reviews.llvm.org/D90507 Files: clang/include/clang/Basic/CodeGenOptions.def clang/include/clang/Driver/Options.td clang/lib/CodeGen

[PATCH] D93656: Moving UniqueInternalLinkageNamesPass to the start of IR pipelines.

aeubanks added inline comments. Comment at: clang/test/CodeGen/unique-internal-linkage-names.cpp:48-49 +// LPIPELINE: Unique Internal Linkage Names +// NPIPELINE: Running pass: UniqueInternalLinkageNamesPass // PLAIN: @_ZL4glob = internal global dblaikie wrote

[PATCH] D93656: Moving UniqueInternalLinkageNamesPass to the start of IR pipelines.

dblaikie added inline comments. Comment at: clang/test/CodeGen/unique-internal-linkage-names.cpp:48-49 +// LPIPELINE: Unique Internal Linkage Names +// NPIPELINE: Running pass: UniqueInternalLinkageNamesPass // PLAIN: @_ZL4glob = internal global aeubanks wrote

[clang] 69132d1 - [Clang] Reverse test to save on indentation. NFC.

Author: Alexandre Ganea Date: 2020-12-23T19:24:53-05:00 New Revision: 69132d12deae749a8e4c9def5498ffa354ce1fa6 URL: https://github.com/llvm/llvm-project/commit/69132d12deae749a8e4c9def5498ffa354ce1fa6 DIFF: https://github.com/llvm/llvm-project/commit/69132d12deae749a8e4c9def5498ffa354ce1fa6.dif

[PATCH] D93785: [OpenMP][FIX] Ensure the isa trait is evaluated last

jdoerfert created this revision. jdoerfert added reviewers: tianshilei1992, jhuber6, JonChesterfield. Herald added subscribers: guansong, bollu, yaxunl. jdoerfert requested review of this revision. Herald added subscribers: llvm-commits, cfe-commits, sstefan1. Herald added projects: clang, LLVM. S

[PATCH] D93786: [OpenMP][Fix] Make the arch selector for x86_64 work

jdoerfert created this revision. jdoerfert added reviewers: tianshilei1992, jhuber6, JonChesterfield. Herald added subscribers: pengfei, guansong, bollu, hiraditya, yaxunl. jdoerfert requested review of this revision. Herald added subscribers: llvm-commits, cfe-commits, sstefan1. Herald added proje

[PATCH] D93656: Moving UniqueInternalLinkageNamesPass to the start of IR pipelines.

hoy added inline comments. Comment at: clang/test/CodeGen/unique-internal-linkage-names.cpp:48-49 +// LPIPELINE: Unique Internal Linkage Names +// NPIPELINE: Running pass: UniqueInternalLinkageNamesPass // PLAIN: @_ZL4glob = internal global dblaikie wrote: > a

[PATCH] D93786: [OpenMP][Fix] Make the arch selector for x86_64 work

tianshilei1992 accepted this revision. tianshilei1992 added a comment. This revision is now accepted and ready to land. Is it because the triple is separated by `-`? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93786/new/ https://reviews.llvm.org/

[PATCH] D93656: Moving UniqueInternalLinkageNamesPass to the start of IR pipelines.

hoy updated this revision to Diff 313630. hoy added a comment. Adding PTO checks in LLVM test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93656/new/ https://reviews.llvm.org/D93656 Files: llvm/include/llvm/Passes/PassBuilder.h llvm/lib/Pass

[PATCH] D93787: [analyzer] Fix crash caused by accessing empty map

vabridgers created this revision. vabridgers added a reviewer: Szelethus. Herald added subscribers: steakhal, ASDenysPetrov, martong, Charusso, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun, whisperity. vabridgers requested review of this revision. Herald a

  1   2   >