[PATCH] D152308: [RISCV] Prevent overflowing the small size of InstSeq in generateInstSeq.
Alex Bradbury via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 12 11:36:22 PDT 2023
asb accepted this revision.
asb added a comment.
This revision is now accepted and ready to land.
Sorry for the slow review, I was mostly out / traveling last week.
The original logic was written to be dumb but easy to follow, and in the absence of benchmarking data I'd be tempted to just bump the size to keep the logic that way. But this change seems correct and I've got not real objection to it if it's your preferred approach.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152308/new/
https://reviews.llvm.org/D152308
More information about the llvm-commits
mailing list