craig.topper added a comment. t
================ Comment at: clang/utils/TableGen/RISCVVEmitter.cpp:58-62 + bool IsPointer = false; + // IsConstant indices are "int", but have the constant expression. + bool IsImmediate = false; + // const qualifier. + bool IsConstant = false; ---------------- jrtc27 wrote: > This isn't expressive enough for the grammar you defined. `PCPCec` is > supposed to give `const i8 * const i8 *`, whereas this will interpret it as > `const i8 *`. Given such types are presumably not needed you need to tighten > the rules of your grammar. @jrtc, are you asking for RVVType::applyModifier to verify that that C doesn't appear twice for example? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95016/new/ https://reviews.llvm.org/D95016 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
