[Lldb-commits] [PATCH] D151003: [Damangle] convert dlangDemangle to use std::string_view

2023-06-05 Thread Eli Friedman via Phabricator via lldb-commits
efriedma added inline comments. Comment at: llvm/lib/Demangle/DLangDemangle.cpp:555 -Demangler D = Demangler(MangledName); -MangledName = D.parseMangle(&Demangled); +Demangler D(MangledName.data()); +const char *M = D.parseMangle(&Demangled); Is

[Lldb-commits] [PATCH] D148761: Emit the correct flags for the PROC CodeView Debug Symbol

2023-05-16 Thread Eli Friedman via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGf8499d5709e3: Emit the correct flags for the PROC CodeView Debug Symbol (authored by dpaoliello, committed by efriedma). Herald added a project: LLDB. Herald added a subscriber: lldb-commits. Changed prio

[Lldb-commits] [PATCH] D140996: [c++20] P1907R1: Support for generalized non-type template arguments of scalar type.

2023-05-14 Thread Eli Friedman via Phabricator via lldb-commits
efriedma added inline comments. Comment at: clang/lib/AST/ItaniumMangle.cpp:4579 + if (CE->hasAPValueResult()) +mangleValueInTemplateArg(ParamType, CE->getResultAsAPValue(), false, + /*NeedExactType=*/true); I'm not su

[Lldb-commits] [PATCH] D129131: Remove uses of llvm_shutdown

2022-08-03 Thread Eli Friedman via Phabricator via lldb-commits
efriedma added inline comments. Comment at: polly/lib/External/isl/interface/extract_interface.cc:590 delete Clang; - llvm::llvm_shutdown(); nhaehnle wrote: > Meinersbur wrote: > > This file is imported from the upstream project > > (https://repo

[Lldb-commits] [PATCH] D127684: [NFC] Use `https` instead of `http` in the urls

2022-06-28 Thread Eli Friedman via Phabricator via lldb-commits
efriedma added a comment. A couple notes skimming the changes: 1. If you're going to update URLs, please verify the new URLs are actually valid. 2. Some of the "URLs" you're updating aren't actually URLs. For example, the `xmlns` attribute in SVG files must be the exact string written in the s

[Lldb-commits] [PATCH] D77587: [SVE] Add new VectorType subclasses

2020-04-09 Thread Eli Friedman via Phabricator via lldb-commits
efriedma added a comment. Looks right, generally. Comment at: llvm/include/llvm-c/Core.h:163 + * value of enum variants after the removal of LLVMVectorTypeKind + */ + LLVMMetadataTypeKind = LLVMPointerTypeKind + 2, /**< Metadata */ The C API is officially