This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGd25c79dc7000: [LoongArch] Support InlineAsm for LSX and LASX
(authored by chenli, committed by SixWeining).
Repository:
rG LLVM Github Monorepo
C
xen0n accepted this revision.
xen0n added a comment.
Seems the `u` and `w` modifiers are strictly an implementation detail that's
different from ordinary inline asm constraints, so even though they take up
precious single-letter namespace I'd be fine.
Given the architecture-specific nature of t
leecheechen updated this revision to Diff 543155.
leecheechen added a comment.
Address @SixWeining's comments.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154931/new/
https://reviews.llvm.org/D154931
Files:
clang/lib/Basic/Targets/LoongArch.cp
SixWeining added inline comments.
Comment at: llvm/lib/Target/LoongArch/LoongArchISelLowering.cpp:56
addRegisterClass(MVT::f64, &LoongArch::FPR64RegClass);
+ if (Subtarget.hasExtLSX()) {
+for (auto VT : {MVT::v4f32, MVT::v2f64, MVT::v16i8, MVT::v8i16, MVT::v4i32,
--
leecheechen created this revision.
Herald added a subscriber: hiraditya.
Herald added a project: All.
leecheechen added reviewers: xen0n, xry111, hev, gonglingqin, SixWeining,
wangleiat.
leecheechen published this revision for review.
Herald added projects: clang, LLVM.
Herald added subscribers: l
