eopXD added inline comments.
================ Comment at: clang/test/Preprocessor/riscv-target-features.c:230 +// RUN: | FileCheck --check-prefix=CHECK-V-MINVLEN %s +// CHECK-V-MINVLEN: __riscv_v_min_vlen 128 ---------------- frasercrmck wrote: > Are we able to test non-default values of `__riscv_v_min_vlen` here? Yes we shall, thank you for the reminder. ================ Comment at: llvm/lib/Target/RISCV/RISCVSubtarget.cpp:119 + if (ZvlLen > RVVVectorBitsMax) + return 0; + // FIXME: Change to >= 32 when VLEN = 32 is supported ---------------- frasercrmck wrote: > Is this intuitive behaviour? If the user supplies `RVVVectorBitsMax` and it's > less than `ZvlLen`, should it silently return? Or do we instead see > `RVVVectorBitsMax` as a user-guided limit //on top// of the architecture? > Which means it can be less but not more? I'm not sure. Added report_fatal_error here. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108694/new/ https://reviews.llvm.org/D108694 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
