It's the same as D126181 for RISCV.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Looks good to me except some nits. @wangleiat Do you have any comments?
llvm/test/CodeGen/LoongArch/eh-dwarf-cfa.ll | ||
---|---|---|
2 | Use double dashes. | |
2 | Ditto. |
llvm/lib/Target/LoongArch/LoongArchISelLowering.cpp | ||
---|---|---|
181 | Incorrect type |
llvm/lib/Target/LoongArch/LoongArchISelLowering.cpp | ||
---|---|---|
181 | Oh my. I was pretty sure I used auto before, not sure how I managed to keep the wrong "bool" type (of IsRISCV64) in the submission... Fixed though. |
llvm/lib/Target/LoongArch/LoongArchISelLowering.cpp | ||
---|---|---|
181 | Seems this is only used one time. Better |
llvm/lib/Target/LoongArch/LoongArchISelLowering.cpp | ||
---|---|---|
181 | In this particular case, if all "values used only once" were to be inlined into the use site, no local values would remain. You would be left with a giant return which probably won't help with readability. (And I already slightly tweaked the code compared to the RISCV version.) So I don't feel very inclined to inline more... |
Incorrect type