craig.topper added inline comments.
================ 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; ---------------- craig.topper wrote: > 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? Oops I meant to write @jrtc27 above. 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
