[PATCH] D150196: [LoongArch] Remove AssemblerPredicate for features: f/d/lsx/lasx/lvz/lbt

Tue May 9 08:08:53 PDT 2023

SixWeining added a comment.

In D150196#4329518 <https://reviews.llvm.org/D150196#4329518>, @xry111 wrote:

> Hmm, I think it's a kernel issue and should be fixed by using `-mabi=lp64s` instead of `-msoft-float` for this specific file...

Makes sense. I'm not sure whether kernel‘s Makefile can distinguish .c and .S (or .c with floating-point insn inlineasm). I'll report this issue to Huacai tomorrow.

Seems there is another approach: add an assembler option to IAS called `-mfpu=64/32/0/none`.
Then kernel Makefile can be:

  if LLVM == 1 && LLVM_IAS ==1
    cflags-y = -msolft-float -Wa,-mfpu=64
  endif

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150196