[PATCH] D121984: [RISCV][NFC] Moving RVV intrinsic type related util to llvm/Support
Fangrui Song via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 28 01:36:36 PDT 2022
MaskRay added a comment.
The original patch added 600 lines while the subject just said "Move". This may be fishy as well.
Having TableGen related `void RVVIntrinsic::emitCodeGenSwitchBody(raw_ostream &OS) const {` in LLVMSupport is also weird.
So I think this patch deserves more discussion and should not be simply reapplied.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121984/new/
https://reviews.llvm.org/D121984
More information about the cfe-commits
mailing list