[PATCH] D143657: [Clang][RISCV] Guard vector float16 type correctly with semantic analysis

Yueh-Ting (eop) Chen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 9 09:30:54 PST 2023


eopXD added a comment.

Forward declaration (snippet below) is still allowed after this commit.

  #include <riscv_vector.h>
  
  vfloat16m1_t foo();

I see the same behavior under https://github.com/llvm/llvm-project/blob/main/clang/test/Sema/x86_64-no-x87.cpp.  If I add type check under `SemaDecl.cpp::Sema::ActOnFunctionDeclarator`, the compiler will emit repeating warning for the same function. Currently I don't know what is the best fix for this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143657



More information about the cfe-commits mailing list