[PATCH] D145088: [RISCV] Add attribute(riscv_rvv_vector_bits(N)) based on AArch64 arm_sve_vector_bits.

[Craig Topper via Phabricator via cfe-commits]

craig.topper added a comment.

In D145088#4307772 <https://reviews.llvm.org/D145088#4307772>, @akyrtzi wrote:

> Hi @craig.topper , this patch is causing a build failure:
>
>   In file included from /llvm-project/clang/lib/Sema/SemaType.cpp:43:
>   /llvm-project/llvm/include/llvm/TargetParser/RISCVTargetParser.h:32:10: fatal error: 'llvm/TargetParser/RISCVTargetParserDef.inc' file not found
>
> To reproduce, configure from a clean build directory like this:
>
>   cmake -G Ninja /path/to/llvm-project/llvm \
>     -DLLVM_TARGETS_TO_BUILD="X86;ARM;AArch64" \
>     -DCMAKE_BUILD_TYPE:STRING=Release \
>     -DLLVM_ENABLE_PROJECTS="clang"
>
> Then run:
>
>   ninja tools/clang/lib/Sema/CMakeFiles/obj.clangSema.dir/SemaType.cpp.o
>
> Could you take a look? 🙏

Thanks, I pushed a4797869e73355209206a5175c11bedb14013211 <https://reviews.llvm.org/rGa4797869e73355209206a5175c11bedb14013211> to fix this. I'm going to find a better home for `RISCV::RVVBitsPerBlock` so we can remove this dependency.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145088