fpetrogalli added a comment. In D80323#2048457 <https://reviews.llvm.org/D80323#2048457>, @rjmccall wrote:
> I'm sympathetic to wanting to get rid of the boolean flag, but this is a > really invasive change for pretty minimal benefit. Why not leave > `VectorType::get` as meaning a non-scalable vector type and come up with a > different method name to get a scalable vector? Sorry @rjmccall , I saw your comments only after approving this patch. I didn't mean to enforce my views over yours. @ctetreau , please make sure you address @rjmccall comment before submitting. Kind regards, Francesco Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80323/new/ https://reviews.llvm.org/D80323 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
