[PATCH] D134798: [LoongArch] Do not assert value type in isFPImmLegal

WÁNG Xuěruì via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 28 04:39:03 PDT 2022


xen0n added inline comments.


================
Comment at: llvm/test/CodeGen/LoongArch/vector-fp-imm.ll:2
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
+; RUN: llc --mtriple=loongarch32 -mattr=+f,-d < %s | FileCheck %s --check-prefix=LA32F
+; RUN: llc --mtriple=loongarch32 -mattr=+d < %s | FileCheck %s --check-prefix=LA32D
----------------
double dash


================
Comment at: llvm/test/CodeGen/LoongArch/vector-fp-imm.ll:7
+
+;; TODO: Merge the offset of address calculation into the offset field of instructions.
+
----------------
What part of the expected output does this comment refer to? I can only see the offsets like 4 8 or 12 nicely placed inside the `f{ld,st}`'s already...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134798



More information about the llvm-commits mailing list