@@ -101,7 +101,7 @@ def extract_result_types(comment):
def strip_doxygen(comment):
-"""Returns the given comment without \-escaped words."""
+"""Returns the given comment without \\-escaped words."""
AngryLoki wrote:
No, `'''\-'''` is still SyntaxWar
https://github.com/kovdan01 updated
https://github.com/llvm/llvm-project/pull/84387
>From 728f5644aebfafd2114e7e47a9b83ef057423997 Mon Sep 17 00:00:00 2001
From: Jonas Devlieghere
Date: Tue, 20 Feb 2024 10:57:54 -0800
Subject: [PATCH 01/10] Upstream ptrauth changes to DWARFASTParserClang
---
https://github.com/kovdan01 updated
https://github.com/llvm/llvm-project/pull/84387
>From 728f5644aebfafd2114e7e47a9b83ef057423997 Mon Sep 17 00:00:00 2001
From: Jonas Devlieghere
Date: Tue, 20 Feb 2024 10:57:54 -0800
Subject: [PATCH 1/6] Upstream ptrauth changes to DWARFASTParserClang
---
..
@@ -664,6 +685,17 @@ CompilerType CompilerType::GetPointerType() const {
return CompilerType();
}
+CompilerType
+CompilerType::AddPtrAuthModifier(unsigned key, bool isAddressDiscriminated,
kovdan01 wrote:
> Also, whose this user of this API?
I've updated t
@@ -216,6 +216,16 @@ class TypeSystem : public PluginInterface,
virtual uint32_t GetPointerByteSize() = 0;
+ // TODO: are we allowed to insert virtual functions in the middle of the
class
+ // interface and break ABI?
kovdan01 wrote:
Thanks, deleted TOD
@@ -676,6 +677,62 @@ DWARFASTParserClang::ParseTypeModifier(const SymbolContext
&sc,
case DW_TAG_volatile_type:
encoding_data_type = Type::eEncodingIsVolatileUID;
break;
+ case DW_TAG_LLVM_ptrauth_type: {
+DWARFDIE ptr_die = die.GetReferencedDIE(DW_AT_type);
---
https://github.com/emaste approved this pull request.
(Assuming the comment removal gets squashed into one commit)
https://github.com/llvm/llvm-project/pull/84147
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/
https://github.com/emaste commented:
It looks like the commit messages and content are mismatched - the commit
message references AT_HWCAP2 but it doesn't look like that's included here?
https://github.com/llvm/llvm-project/pull/85057
___
lldb-commits
https://github.com/rupprecht created
https://github.com/llvm/llvm-project/pull/86962
The commit 8bed754c2f965c8cbbb050be6f650b78f7fd78a6 was intended to support the
use case where users want to run all the LLDB tests in an environment where
pexpect is not installed. Those users can build with
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Jordan Rupprecht (rupprecht)
Changes
The commit 8bed754c2f965c8cbbb050be6f650b78f7fd78a6 was intended to support the
use case where users want to run all the LLDB tests in an environment where
pexpect is not installed. Those users can buil
https://github.com/JDevlieghere approved this pull request.
Works for us!
https://github.com/llvm/llvm-project/pull/86962
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
kovdan01 wrote:
@jasonmolenda Would be glad to see your feedback - see answers to your previous
comments above
https://github.com/llvm/llvm-project/pull/82603
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
Author: Jordan Rupprecht
Date: 2024-03-28T15:57:09-05:00
New Revision: a8340767de71b8c7d10de74019622592b39e3704
URL:
https://github.com/llvm/llvm-project/commit/a8340767de71b8c7d10de74019622592b39e3704
DIFF:
https://github.com/llvm/llvm-project/commit/a8340767de71b8c7d10de74019622592b39e3704.di
https://github.com/rupprecht closed
https://github.com/llvm/llvm-project/pull/86962
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/cmtice updated
https://github.com/llvm/llvm-project/pull/86862
>From 03ef6db02f49ab4cc4f63b32553015b4f3801b5b Mon Sep 17 00:00:00 2001
From: Caroline Tice
Date: Wed, 27 Mar 2024 10:09:45 -0700
Subject: [PATCH 1/4] [LLDB] Add APFloat helper functions to Scalar class.
This add
cmtice wrote:
> Could you add a test for these new methods in
> `unittests/Utility/ScalarTest.cpp`?
Done. :-)
https://github.com/llvm/llvm-project/pull/86862
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
https://github.com/JDevlieghere approved this pull request.
https://github.com/llvm/llvm-project/pull/86862
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/bulbazord approved this pull request.
https://github.com/llvm/llvm-project/pull/86862
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
bulbazord wrote:
> > Could you add a test for these new methods in
> > `unittests/Utility/ScalarTest.cpp`?
>
> Done. :-)
Thank you! :)
https://github.com/llvm/llvm-project/pull/86862
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://
Author: cmtice
Date: 2024-03-28T21:39:58-07:00
New Revision: ba6b2d22af177a72b132cdb8e9350a708f282d2c
URL:
https://github.com/llvm/llvm-project/commit/ba6b2d22af177a72b132cdb8e9350a708f282d2c
DIFF:
https://github.com/llvm/llvm-project/commit/ba6b2d22af177a72b132cdb8e9350a708f282d2c.diff
LOG: [
https://github.com/cmtice closed https://github.com/llvm/llvm-project/pull/86862
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
21 matches
Mail list logo
