[PATCH] D152192: [RISCV] Make .option arch parser less mind-bending
Jessica Clarke via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 5 19:25:17 PDT 2023
jrtc27 added inline comments.
================
Comment at: llvm/lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp:2745
+
+ if (IsFull) {
std::string Result;
----------------
craig.topper wrote:
> craig.topper wrote:
> > I think you could use `!IsExtensionList` here and avoid the `IsFull` flag. That might make the code less readable, but maybe that can be fixed with renaming `IsExtensionList`
> Nevermind, I see the followup patch changes this to remove IsExtensionList
I guess I could still as the interim state rather than introduce an unnecessary extra variable, though probably not worth the hassle of changing this revision for code that's going to immediately be deleted
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152192/new/
https://reviews.llvm.org/D152192
More information about the llvm-commits
mailing list