[PATCH] D120476: [LoongArch] Add basic support to AsmParser

Min-Yih Hsu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 25 15:05:03 PST 2022


myhsu added a comment.

> The previous .mir test uses incorrect MIR
> Here, 34 is an illegal immediate operand for the BL instruction. The reason why it doesn't fails is getImmOpValueAsr2 is not added at that time.

That makes more sense. Just make sure you really fix it in this patch.
Thanks for the clarifications.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120476



More information about the llvm-commits mailing list