eopXD added inline comments.
================ Comment at: llvm/lib/Target/RISCV/RISCVSubtarget.h:141 + // either v or zve* suppaort v instructions + bool hasStdExtV() const { return HasStdExtV || HasStdExtZve32x; } + bool hasStdExtZve32x() const { return HasStdExtZve32x; } ---------------- craig.topper wrote: > frasercrmck wrote: > > Is this correct? I thought we'd keep `hasStdExtV` as being the > > single-letter V extension, and Zve32x isn't that. > I just put up D112496 to stop using hasStdExtV everywhere. Thank you @craig.topper for the patch. LGTM, and we can have it landed after this series on `zvl`, `zve` patch are done. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112408/new/ https://reviews.llvm.org/D112408 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits