[libcxx-commits] [PATCH] D72952: [libunwind] Set LIBUNWIND_ASM_SOURCES to the ASM source language from C

Shoaib Meenai via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Jan 17 15:58:59 PST 2020


smeenai added a comment.

In D72952#1827678 <https://reviews.llvm.org/D72952#1827678>, @JamesNagurne wrote:

> In D72952#1827668 <https://reviews.llvm.org/D72952#1827668>, @smeenai wrote:
>
> > The diff here looks weird ... it looks like a diff against your previous diff instead of a diff against master? (Phabricator says the line you're deleting has `LANGUAGE ASM` instead of `LANGUAGE C`, as it does on master.)
>
>
> Eek, sorry. We use CodeCollaborator internally, which would have effectively shown the diff from the original commit to the final (after 2 change sets), which would have shown the C in the before and nothing after. I mistakenly treated Phabricator the same way erroneously.
>
> I'll re-upload this so it looks right. My apologies!


No worries! LGTM.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72952/new/

https://reviews.llvm.org/D72952





More information about the libcxx-commits mailing list