Page MenuHomePhabricator

avogelsgesang (Adrian Vogelsgesang)
User

Projects

User does not belong to any projects.

User Details

User Since
Jul 13 2021, 10:17 AM (72 w, 1 d)

Recent Activity

Today

avogelsgesang added inline comments to D131395: [libc++] Implement `lexicographical_compare_three_way`.
Wed, Nov 30, 7:05 PM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D131395: [libc++] Implement `lexicographical_compare_three_way`.

address varconst's commments

Wed, Nov 30, 7:05 PM · Restricted Project, Restricted Project
avogelsgesang added a comment to D132815: [LLDB] Do not dereference promise pointer in `coroutine_handle` pretty printer.

The issue with this change was that if devirtualization is failing, then in the line

Wed, Nov 30, 1:24 AM · Restricted Project, Restricted Project
avogelsgesang updated subscribers of D132735: [LLDB] Recognize `std::noop_coroutine()` in `std::coroutine_handle` pretty printer.

As discussed via email, the reason for the test failures on green dragon is, that green dragon seems to be using some compiler which passes the test

Wed, Nov 30, 1:21 AM · Restricted Project, Restricted Project

Yesterday

avogelsgesang added inline comments to D131395: [libc++] Implement `lexicographical_compare_three_way`.
Tue, Nov 29, 3:37 PM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D131395: [libc++] Implement `lexicographical_compare_three_way`.
  • forward -> move
  • move iterator initialization out of benchmarking loop
  • remove #if _LIB_CPP_VERSION check from benchmark
Tue, Nov 29, 3:36 PM · Restricted Project, Restricted Project

Fri, Nov 25

avogelsgesang added a comment to D132624: [LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`.

@jasonmolenda I am a bit confused about your revert.
Did you indeed revert this change here? Afaict, you reverted https://reviews.llvm.org/D132735 and https://reviews.llvm.org/D132815, but left this commit in place. I did not actually read through your code changes, though, I only looked at the commit messages so far. Is my understanding of which commits you actually reverted correct?

Fri, Nov 25, 1:48 PM · Restricted Project, Restricted Project

Thu, Nov 24

avogelsgesang updated the diff for D131395: [libc++] Implement `lexicographical_compare_three_way`.
  • sort includes correctly
  • work around NASTY_MACRO
Thu, Nov 24, 5:01 PM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D131395: [libc++] Implement `lexicographical_compare_three_way`.
  • use explicit types instead of auto
  • make benchmark runnable on non-libcxx
Thu, Nov 24, 1:19 PM · Restricted Project, Restricted Project

Wed, Nov 23

avogelsgesang added a comment to D134010: [C++] [Coroutines] Introduce `coro_always_done` attribute to help optimizations (2/2).

And now, it should be clear that why the coro_always_done and coro_always_resume is helpful for coroutine elision. Since it is helpful for such cases.

Wed, Nov 23, 7:28 AM · Restricted Project

Tue, Nov 22

avogelsgesang added a comment to D134010: [C++] [Coroutines] Introduce `coro_always_done` attribute to help optimizations (2/2).

Interesting... . I will need to read up on coro_always_elide a bit before being able to form an informed opinion.
I found https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2022/p2594r0.pdf
Any other relevant resources?

Tue, Nov 22, 4:23 AM · Restricted Project

Mon, Nov 21

avogelsgesang updated the diff for D131395: [libc++] Implement `lexicographical_compare_three_way`.

Another try at fixing the CI; this time the benchmarks

Mon, Nov 21, 1:21 PM · Restricted Project, Restricted Project

Sun, Nov 20

avogelsgesang abandoned D116290: [clang-format] Add enforcement of consistent `class`/typename` keyword for template arguments.
Sun, Nov 20, 6:51 PM · Restricted Project, Restricted Project, Restricted Project
avogelsgesang updated the diff for D132268: [libc++][spaceship] Implement `operator<=>` for `vector`.

add missing _LIBCPP_HIDE_FROM_ABI; use new test utility

Sun, Nov 20, 6:50 PM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D132312: [libc++][spaceship] Implement `operator<=>` for `list`.

add inline

Sun, Nov 20, 6:45 PM · Restricted Project, Restricted Project
avogelsgesang added inline comments to D132312: [libc++][spaceship] Implement `operator<=>` for `list`.
Sun, Nov 20, 6:43 PM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D132312: [libc++][spaceship] Implement `operator<=>` for `list`.

Introduce reusable test function

Sun, Nov 20, 6:43 PM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D131395: [libc++] Implement `lexicographical_compare_three_way`.

fix CI... maybe...

Sun, Nov 20, 5:08 PM · Restricted Project, Restricted Project
avogelsgesang committed rGcd3091a88f7c: [LLDB] Do not dereference promise pointer in `coroutine_handle` pretty printer (authored by avogelsgesang).
[LLDB] Do not dereference promise pointer in `coroutine_handle` pretty printer
Sun, Nov 20, 2:29 PM · Restricted Project
avogelsgesang closed D132815: [LLDB] Do not dereference promise pointer in `coroutine_handle` pretty printer.
Sun, Nov 20, 2:29 PM · Restricted Project, Restricted Project
avogelsgesang committed rG4346318f5c70: [LLDB] Recognize `std::noop_coroutine()` in `std::coroutine_handle` pretty… (authored by avogelsgesang).
[LLDB] Recognize `std::noop_coroutine()` in `std::coroutine_handle` pretty…
Sun, Nov 20, 11:19 AM · Restricted Project
avogelsgesang closed D132735: [LLDB] Recognize `std::noop_coroutine()` in `std::coroutine_handle` pretty printer.
Sun, Nov 20, 11:19 AM · Restricted Project, Restricted Project
avogelsgesang committed rGfb61dce1adc4: [lldb] Fix test expectation in `TestCoroutineHandle.py` for 32-bit systems (authored by avogelsgesang).
[lldb] Fix test expectation in `TestCoroutineHandle.py` for 32-bit systems
Sun, Nov 20, 10:31 AM · Restricted Project
avogelsgesang committed rG01f4c305fae9: Reapply "[LLDB] Devirtualize coroutine promise types for `std… (authored by avogelsgesang).
Reapply "[LLDB] Devirtualize coroutine promise types for `std…
Sun, Nov 20, 6:36 AM · Restricted Project
avogelsgesang closed D132624: [LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`.
Sun, Nov 20, 6:35 AM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D131395: [libc++] Implement `lexicographical_compare_three_way`.

fix CI

Sun, Nov 20, 5:50 AM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D131395: [libc++] Implement `lexicographical_compare_three_way`.

fix CI

Sun, Nov 20, 4:26 AM · Restricted Project, Restricted Project

Sat, Nov 19

avogelsgesang added inline comments to D136268: [libc++][ranges] implement `std::views::elements_view`.
Sat, Nov 19, 9:11 PM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D131395: [libc++] Implement `lexicographical_compare_three_way`.

fix CI

Sat, Nov 19, 9:03 PM · Restricted Project, Restricted Project
avogelsgesang planned changes to D132268: [libc++][spaceship] Implement `operator<=>` for `vector`.

needs to be adjusted to use the still-to-be-introduced test utilities from https://reviews.llvm.org/D132312 is done

Sat, Nov 19, 8:59 PM · Restricted Project, Restricted Project
avogelsgesang planned changes to D132265: [libc++][spaceship] Implement `operator<=>` for `array`.

needs to be adjusted to use the still-to-be-introduced test utilities from https://reviews.llvm.org/D132312 is done

Sat, Nov 19, 8:56 PM · Restricted Project, Restricted Project
avogelsgesang added inline comments to D132265: [libc++][spaceship] Implement `operator<=>` for `array`.
Sat, Nov 19, 8:55 PM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D131395: [libc++] Implement `lexicographical_compare_three_way`.

fix CI

Sat, Nov 19, 8:35 PM · Restricted Project, Restricted Project
avogelsgesang added a comment to D131395: [libc++] Implement `lexicographical_compare_three_way`.

I think I addressed all actionable review comments now.

Sat, Nov 19, 8:30 PM · Restricted Project, Restricted Project
avogelsgesang added inline comments to D131395: [libc++] Implement `lexicographical_compare_three_way`.
Sat, Nov 19, 7:57 PM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D131395: [libc++] Implement `lexicographical_compare_three_way`.

Address mordante's comments about test cases:

Sat, Nov 19, 7:56 PM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D131395: [libc++] Implement `lexicographical_compare_three_way`.

Updated benchmark code to directly compare the slow and the fast path

Sat, Nov 19, 5:34 PM · Restricted Project, Restricted Project
avogelsgesang added a comment to D131395: [libc++] Implement `lexicographical_compare_three_way`.

@var-const As requested, I updated the benchmark to benchmark int* on both the fast and the slow path.

Sat, Nov 19, 5:30 PM · Restricted Project, Restricted Project
avogelsgesang added a comment to D134010: [C++] [Coroutines] Introduce `coro_always_done` attribute to help optimizations (2/2).

Sorry for the late reply - and thanks for your detailed answers!

Sat, Nov 19, 9:46 AM · Restricted Project

Mon, Nov 14

avogelsgesang added a comment to D131938: [C++20] [Coroutines] Disable to take the address of labels in coroutines.

Personally, I am fine with this change as in review right now.

Mon, Nov 14, 7:06 PM · Restricted Project, Restricted Project
avogelsgesang added a comment to D132624: [LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`.

@aprantl I still had to adjust the code slightly to copy the type from the ASTContext for '.../a.out' to the scratch ASTContext (without this, my code was triggering an assert).
See https://reviews.llvm.org/D132624?vs=474793&id=475327 for the related changes.
Because I have no prior experience with ClangASTImporter and the usage of multiple different TypeSystemClang instances, I would appreciate if you could take another quick look

Mon, Nov 14, 6:57 PM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D132624: [LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`.

Fix ownership of CompilerType between "scratch typesystem" and "module type system"

Mon, Nov 14, 6:17 PM · Restricted Project, Restricted Project
avogelsgesang reopened D132624: [LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`.
Mon, Nov 14, 6:17 PM · Restricted Project, Restricted Project
avogelsgesang added a comment to D137893: [AsmWriter] Do not write a comma when varargs is the only argument.

Pushed upstream. Thank you for the fix!

Mon, Nov 14, 2:47 AM · Restricted Project, Restricted Project
avogelsgesang committed rG8ad88f129cc4: Do not write a comma when varargs is the only argument (authored by caojoshua).
Do not write a comma when varargs is the only argument
Mon, Nov 14, 2:47 AM · Restricted Project, Restricted Project
avogelsgesang closed D137893: [AsmWriter] Do not write a comma when varargs is the only argument.
Mon, Nov 14, 2:46 AM · Restricted Project, Restricted Project

Sun, Nov 13

avogelsgesang added a comment to D137893: [AsmWriter] Do not write a comma when varargs is the only argument.

Can you provide me a git patch, including the meta data (email address, ...)?
Not sure how to get this from Differential, I am rather inexperienced with it myself...

Sun, Nov 13, 5:44 PM · Restricted Project, Restricted Project

Sat, Nov 12

avogelsgesang accepted D137893: [AsmWriter] Do not write a comma when varargs is the only argument.

LGTM, with small nit:

Sat, Nov 12, 2:48 PM · Restricted Project, Restricted Project

Fri, Nov 11

avogelsgesang added a reverting change for D132624: [LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`: rGd7e1c2770fa5: Revert "[LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`".
Fri, Nov 11, 10:01 AM · Restricted Project, Restricted Project
avogelsgesang added a reverting change for rG558db7787005: [LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`: rGd7e1c2770fa5: Revert "[LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`".
Fri, Nov 11, 10:01 AM · Restricted Project
avogelsgesang committed rGd7e1c2770fa5: Revert "[LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`" (authored by avogelsgesang).
Revert "[LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`"
Fri, Nov 11, 10:01 AM · Restricted Project
avogelsgesang committed rG558db7787005: [LLDB] Devirtualize coroutine promise types for `std::coroutine_handle` (authored by avogelsgesang).
[LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`
Fri, Nov 11, 9:38 AM · Restricted Project
avogelsgesang closed D132624: [LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`.
Fri, Nov 11, 9:38 AM · Restricted Project, Restricted Project

Fri, Nov 4

avogelsgesang added a comment to D132624: [LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`.

@labath @aprantl Thank you for your reviews on https://reviews.llvm.org/D132815 and https://reviews.llvm.org/D132735
I would appreciate if you also have time to review this change here, as it is a pre-requisite for the other two changes :)

Fri, Nov 4, 4:56 AM · Restricted Project, Restricted Project
avogelsgesang planned changes to D131395: [libc++] Implement `lexicographical_compare_three_way`.

@avogelsgesang Do you need anything from us to make progress on this patch? I'd like to make sure you're unblocked.

Fri, Nov 4, 4:39 AM · Restricted Project, Restricted Project

Sep 20 2022

avogelsgesang added a comment to D134019: [MachineCycle][NFC]add a cache for block and top level cycle mapping.

does it make sense to add a regression test?

Sep 20 2022, 6:22 PM · Restricted Project, Restricted Project
avogelsgesang added inline comments to D134010: [C++] [Coroutines] Introduce `coro_always_done` attribute to help optimizations (2/2).
Sep 20 2022, 8:28 AM · Restricted Project
avogelsgesang added inline comments to D134010: [C++] [Coroutines] Introduce `coro_always_done` attribute to help optimizations (2/2).
Sep 20 2022, 4:35 AM · Restricted Project
avogelsgesang added inline comments to D134269: [docs] Document the one-phase style compilation to c++ standard modules.
Sep 20 2022, 4:12 AM · Restricted Project, Restricted Project, Restricted Project
avogelsgesang added inline comments to D134009: [Coroutines] Introduce `always_complete_coroutine` attribute to guide optimization (1/2).
Sep 20 2022, 3:46 AM · Restricted Project, Restricted Project

Sep 19 2022

avogelsgesang updated the diff for D132735: [LLDB] Recognize `std::noop_coroutine()` in `std::coroutine_handle` pretty printer.

rename IsNoopResumeDestroy -> IsNoopCoroFunction

Sep 19 2022, 9:39 AM · Restricted Project, Restricted Project
avogelsgesang added inline comments to D132815: [LLDB] Do not dereference promise pointer in `coroutine_handle` pretty printer.
Sep 19 2022, 9:28 AM · Restricted Project, Restricted Project
avogelsgesang added a comment to D132815: [LLDB] Do not dereference promise pointer in `coroutine_handle` pretty printer.

wondering if we couldn't fix this by creating the (non-pointer) object using the CreateValueObjectFromAddress function, as above, but then actually use valobj->AddressOf as the synthetic child

Sep 19 2022, 9:27 AM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D132815: [LLDB] Do not dereference promise pointer in `coroutine_handle` pretty printer.

address review comments

Sep 19 2022, 9:25 AM · Restricted Project, Restricted Project
avogelsgesang added inline comments to D134009: [Coroutines] Introduce `always_complete_coroutine` attribute to guide optimization (1/2).
Sep 19 2022, 6:51 AM · Restricted Project, Restricted Project
avogelsgesang added a comment to D134010: [C++] [Coroutines] Introduce `coro_always_done` attribute to help optimizations (2/2).

took a look at the documentation; didn't read the implementation itself

Sep 19 2022, 5:52 AM · Restricted Project

Sep 17 2022

avogelsgesang added a comment to D134009: [Coroutines] Introduce `always_complete_coroutine` attribute to guide optimization (1/2).

Agree, not a blocker

Sep 17 2022, 9:53 AM · Restricted Project, Restricted Project

Sep 16 2022

avogelsgesang added a comment to D134009: [Coroutines] Introduce `always_complete_coroutine` attribute to guide optimization (1/2).

high-level design comment (I didn't look at the code; wouldn't understand it anyway):
I think there is another important case to cover, besides "a coroutine can only be destroyed after it ran to its final suspension point":
"A coroutine can only be destroy after it ran to its final suspension point *or* before it executed until after its initial suspension point"

Sep 16 2022, 1:33 PM · Restricted Project, Restricted Project
avogelsgesang added a comment to D134019: [MachineCycle][NFC]add a cache for block and top level cycle mapping.

Thanks for the quick fix!
I unfortunately don't understand this code here, and can't give you a review, though

Sep 16 2022, 6:38 AM · Restricted Project, Restricted Project
avogelsgesang added a comment to D133846: [libunwind] Fix usage of `_dl_find_object` on 32-bit x86.

Thanks for the quick review, @MaskRay!

Sep 16 2022, 6:37 AM · Restricted Project, Restricted Project, Restricted Project
avogelsgesang committed rGc9cffdde393f: [libunwind] Fix usage of `_dl_find_object` on 32-bit x86 (authored by avogelsgesang).
[libunwind] Fix usage of `_dl_find_object` on 32-bit x86
Sep 16 2022, 6:30 AM · Restricted Project
avogelsgesang closed D133846: [libunwind] Fix usage of `_dl_find_object` on 32-bit x86.
Sep 16 2022, 6:30 AM · Restricted Project, Restricted Project, Restricted Project
avogelsgesang updated the summary of D133846: [libunwind] Fix usage of `_dl_find_object` on 32-bit x86.
Sep 16 2022, 6:29 AM · Restricted Project, Restricted Project, Restricted Project

Sep 15 2022

avogelsgesang updated subscribers of D133846: [libunwind] Fix usage of `_dl_find_object` on 32-bit x86.

My understanding is, that I am not supposed to merge to main until after I got approval from the libunwind review group.
Not sure how the priority of following those process requirements compare to the unblocking Gentoo testing. I am too new to LLVM to really make a call here...

Sep 15 2022, 11:37 AM · Restricted Project, Restricted Project, Restricted Project
avogelsgesang added reviewers for D133846: [libunwind] Fix usage of `_dl_find_object` on 32-bit x86: MaskRay, compnerd.

@MaskRay and @compnerd could I get a review from you on this change, please :)

Sep 15 2022, 7:04 AM · Restricted Project, Restricted Project, Restricted Project
avogelsgesang added a comment to D133846: [libunwind] Fix usage of `_dl_find_object` on 32-bit x86.

Thanks for those build instructions! I was able to reproduce the test failures.
The updated review now fixes the failures for me. Could you please double-check if it also fixes the failures for you?

Sep 15 2022, 5:28 AM · Restricted Project, Restricted Project, Restricted Project
avogelsgesang updated the summary of D133846: [libunwind] Fix usage of `_dl_find_object` on 32-bit x86.
Sep 15 2022, 5:27 AM · Restricted Project, Restricted Project, Restricted Project
avogelsgesang updated the diff for D133846: [libunwind] Fix usage of `_dl_find_object` on 32-bit x86.

remove assert completely

Sep 15 2022, 5:26 AM · Restricted Project, Restricted Project, Restricted Project

Sep 14 2022

avogelsgesang added a comment to D133846: [libunwind] Fix usage of `_dl_find_object` on 32-bit x86.

Thanks for checking this, @mgorny!
Well, I guess I will have to dig deeper, then...
Can you share a (preferably minimal) command line how to reproduce the test failures on a 64-bit Linux machine?

Sep 14 2022, 7:41 AM · Restricted Project, Restricted Project, Restricted Project
avogelsgesang requested review of D133846: [libunwind] Fix usage of `_dl_find_object` on 32-bit x86.
Sep 14 2022, 3:42 AM · Restricted Project, Restricted Project, Restricted Project

Sep 9 2022

avogelsgesang added a comment to D123995: [MachineSink] replace MachineLoop with MachineCycle.

FYI: I suspect this change caused a compile-time regression from 60sec to >40 minutes for an interpreter-loop based on computed-gotos. See https://github.com/llvm/llvm-project/issues/57664

Sep 9 2022, 4:41 PM · Restricted Project, Restricted Project

Sep 6 2022

avogelsgesang added a comment to D126122: Fix RUNPATH not accounting for LLVM_ENABLE_PER_TARGET_RUNTIME_DIR.

for what it's worth: As part of upgrading the Salesforce-internal clang toolchain to clang 15.0.0, I also ran into the RUNPATH issues. This patch here solved those issues for us

Sep 6 2022, 4:47 AM · Restricted Project, Restricted Project

Sep 5 2022

avogelsgesang accepted D133323: [libc++] Bump _LIBCPP_STD_VER to the next expected C++ version.

Are you planning to also replace existing usages of, e.g. #if _LIBCPP_STD_VER > 17 by #if _LIBCPP_STD_VER >= 20?

Sep 5 2022, 4:03 PM · Restricted Project, Restricted Project

Aug 31 2022

avogelsgesang added inline comments to D132815: [LLDB] Do not dereference promise pointer in `coroutine_handle` pretty printer.
Aug 31 2022, 7:56 AM · Restricted Project, Restricted Project
avogelsgesang added inline comments to D132815: [LLDB] Do not dereference promise pointer in `coroutine_handle` pretty printer.
Aug 31 2022, 7:04 AM · Restricted Project, Restricted Project

Aug 28 2022

avogelsgesang added inline comments to D131395: [libc++] Implement `lexicographical_compare_three_way`.
Aug 28 2022, 2:00 PM · Restricted Project, Restricted Project
avogelsgesang added a comment to D131963: [libc++] Add custom clang-tidy checks.

is there a reason why we add our own clang-tidy plugin instead of upstreaming this to clang-tidy itself? I could imagine that other library developers could also benefit from this clang-tidy check, if we make the list of allowed customization points a configuration option

Aug 28 2022, 1:35 PM · Restricted Project, Restricted Project, Restricted Project
avogelsgesang added a comment to D132624: [LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`.

@labath D132815 addresses your point: With that change, the promise member now is typed as a pointer.

Aug 28 2022, 1:13 PM · Restricted Project, Restricted Project
avogelsgesang requested review of D132815: [LLDB] Do not dereference promise pointer in `coroutine_handle` pretty printer.
Aug 28 2022, 1:08 PM · Restricted Project, Restricted Project

Aug 27 2022

avogelsgesang updated the diff for D132735: [LLDB] Recognize `std::noop_coroutine()` in `std::coroutine_handle` pretty printer.

fix test case

Aug 27 2022, 3:35 PM · Restricted Project, Restricted Project

Aug 26 2022

avogelsgesang updated the diff for D132735: [LLDB] Recognize `std::noop_coroutine()` in `std::coroutine_handle` pretty printer.

fix offset

Aug 26 2022, 4:43 PM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D132735: [LLDB] Recognize `std::noop_coroutine()` in `std::coroutine_handle` pretty printer.

rebase

Aug 26 2022, 4:38 PM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D132624: [LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`.

disable tests on gcc

Aug 26 2022, 4:19 PM · Restricted Project, Restricted Project
avogelsgesang added inline comments to D131395: [libc++] Implement `lexicographical_compare_three_way`.
Aug 26 2022, 2:19 PM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D131395: [libc++] Implement `lexicographical_compare_three_way`.

address more comments, but now gcc11 is crashing.
updating so that CI can hopefully give me some insight into whether it's fixed in gcc12.

Aug 26 2022, 2:03 PM · Restricted Project, Restricted Project
avogelsgesang added a comment to D132624: [LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`.

I don't know much about coroutines, but it seems like your goal is to format them like a linked list

Aug 26 2022, 7:05 AM · Restricted Project, Restricted Project
avogelsgesang added a comment to D132624: [LLDB] Devirtualize coroutine promise types for `std::coroutine_handle`.

I don't know much about coroutines, but it seems like your goal is to format them like a linked list

Aug 26 2022, 6:57 AM · Restricted Project, Restricted Project
avogelsgesang updated the diff for D132735: [LLDB] Recognize `std::noop_coroutine()` in `std::coroutine_handle` pretty printer.

Exclude unsupported compilers from test case

Aug 26 2022, 6:43 AM · Restricted Project, Restricted Project
avogelsgesang added a comment to D132580: [Coro][Debuginfo] Add debug info to `_NoopCoro_ResumeDestroy` function.

merged to main despite test failures because those test failures were only timeouts and I don't see how those could be related to this change at hand here

Aug 26 2022, 5:51 AM · debug-info, Restricted Project, Restricted Project
avogelsgesang committed rG5af06ba7dc26: [Coro][Debuginfo] Add debug info to `__NoopCoro_ResumeDestroy` function (authored by avogelsgesang).
[Coro][Debuginfo] Add debug info to `__NoopCoro_ResumeDestroy` function
Aug 26 2022, 5:50 AM · Restricted Project, Restricted Project