[PATCH] D53146: [MC][ELF] fix newly added test

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 11 11:18:45 PDT 2018


nickdesaulniers added a subscriber: MaskRay.
nickdesaulniers added a comment.

> but the asm syntax was different
>  Oh, you've done that. The diff isn't clear.

@MaskRay fixed this mistake of mine in r344206.

> I don't think CodeGen/Generic is the right place. Why can't you just leave in MC/ELF with the new CHECK line?

I agree; note that phabricator has funny UI for a moved file that was then modified.  Triple check the direction of the arrows (-->, <--); note how they flip for the modification.


Repository:
  rL LLVM

https://reviews.llvm.org/D53146





More information about the llvm-commits mailing list