hvdijk added a comment.

In D100148#2683325 <https://reviews.llvm.org/D100148#2683325>, @glaubitz wrote:

> Hi! Can we get this patch merged as-is or do we need anything else?

Sorry, miscommunication. I was thinking you could use this to update D99988 
<https://reviews.llvm.org/D99988> to actually build the relevant compiler-rt 
bits for x32, and then once both that and this are ready, both can be submitted 
at the same time. The reason for that is that this is not useful by itself, and 
if it turns out that D99988 <https://reviews.llvm.org/D99988> actually requires 
some other clang change as well that we are not seeing yet, it will be easier 
to update this if it has not been pushed yet. Do you think it would be better 
to push this first?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100148

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D100148:... Harald van Dijk via Phabricator via cfe-commits
    • [PATCH] D10... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D10... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D10... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D10... Harald van Dijk via Phabricator via cfe-commits
    • [PATCH] D10... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D10... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D10... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D10... Harald van Dijk via Phabricator via cfe-commits
    • [PATCH] D10... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D10... Harald van Dijk via Phabricator via cfe-commits
    • [PATCH] D10... John Paul Adrian Glaubitz via Phabricator via cfe-commits

Reply via email to