[all-commits] [llvm/llvm-project] 865fb2: [RISCV] Pre-commit test case for D140460
Jim Lin via All-commits
all-commits at lists.llvm.org
Thu Mar 9 18:25:43 PST 2023
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 865fb2e44011c60d6d69154d205b55f4b44ff10b
https://github.com/llvm/llvm-project/commit/865fb2e44011c60d6d69154d205b55f4b44ff10b
Author: Jim Lin <jim at andestech.com>
Date: 2023-03-10 (Fri, 10 Mar 2023)
Changed paths:
A llvm/test/CodeGen/RISCV/stack-slot-coloring.mir
Log Message:
-----------
[RISCV] Pre-commit test case for D140460
Reviewed By: craig.topper
Differential Revision: https://reviews.llvm.org/D145549
Commit: 7e9293572d332b812eb1e521f0a75a4e7034abbf
https://github.com/llvm/llvm-project/commit/7e9293572d332b812eb1e521f0a75a4e7034abbf
Author: Jim Lin <jim at andestech.com>
Date: 2023-03-10 (Fri, 10 Mar 2023)
Changed paths:
M llvm/lib/Target/RISCV/RISCVInstrInfo.cpp
M llvm/lib/Target/RISCV/RISCVInstrInfo.h
M llvm/test/CodeGen/RISCV/stack-slot-coloring.mir
Log Message:
-----------
[RISCV] Set how many bytes load from or store to stack slot
Refer from: https://reviews.llvm.org/D44782
After https://reviews.llvm.org/D130302, LW+SEXT.B can be folded into LB
as partially reload stack slot. This gains incorrect optimization result
from `StackSlotColoring` without given the number of bytes exactly load
from stack. LB+SW are mis-interpreted as fully reload/restore from stack
slot without the sign-extension. SW would be considered as a redundant store.
The testcase is copied from llvm/test/CodeGen/X86/pr30821.mir.
Reviewed By: craig.topper
Differential Revision: https://reviews.llvm.org/D145471
Compare: https://github.com/llvm/llvm-project/compare/bb01b9919fdd...7e9293572d33
More information about the All-commits
mailing list