[PATCH] D77118: [PowerPC] Ignore implicit register operands for MCInst

Zhang Kang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 31 09:58:13 PDT 2020


ZhangKang marked 4 inline comments as done.
ZhangKang added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/atomics-regression.ll:1234
 ; PPC64LE-NEXT:    sync
-; PPC64LE-NEXT:  .LBB79_1:
+; PPC64LE-NEXT:  .LBB79_1: #
 ; PPC64LE-NEXT:    ldarx 6, 0, 3
----------------
jsji wrote:
> ZhangKang wrote:
> > steven.zhang wrote:
> > > Could you please explain more on the extra "#" here ?
> > This case is updated by update_llc_test_checks.py, last update is in Aug 30, 2019. After last update, someone modify the assembly output to add extra `#`.
> Can you please find out the patch? If the change is reasonable, then we should commit a NFC patch to update the testcases first, then rebase.
This is caused by https://reviews.llvm.org/D63957 ,it will be fixed soon.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77118





More information about the llvm-commits mailing list