[PATCH] D99751: Add the TableGen assert statement, step 3

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 8 10:48:51 PDT 2021


dblaikie added a comment.

In D99751#2677328 <https://reviews.llvm.org/D99751#2677328>, @Paul-C-Anagnostopoulos wrote:

> The reason I haven't reverted this patch is because I don't feel comfortable doing a revert with the tangled up branches I have. Do you think it's safe to revert the commit on my 'assert' branch even though it is not the topmost commit?

I'm not sure what state your branches are in, so I can't say for sure - hence the suggestion to start a new/separate/clean git checkout and revert it there (you could then throw away that checkout, or you could port the patches from your tangled up checkout instead of fixing it (admittedly fixing it's probably a good experience to figure out what went wrong/how to fix it/etc - but it's not uncommon for folks (myself included) to sometimes just blow things away and start again (porting patches by hand) if it's gotten all messed up))


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99751



More information about the llvm-commits mailing list