Splitting this out from D29934, as requested during review.
This patch adds support for lowering global addresses and tests for load/store on those addresses.
Paths
| Differential D39103
[RISCV 12.5/n] Codegen support for memory operations on global addresses ClosedPublic Authored by asb on Oct 19 2017, 12:21 PM.
Details Summary Splitting this out from D29934, as requested during review. This patch adds support for lowering global addresses and tests for load/store on those addresses.
Diff Detail Event TimelineComment Actions I accidentally ttached an older version of the patch before, which had an unwanted whitespace change. Comment Actions LGTM
This revision is now accepted and ready to land.Oct 20 2017, 4:02 PM Closed by commit rL317688: [RISCV] Codegen support for memory operations on global addresses (authored by asb). · Explain WhyNov 8 2017, 5:25 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 119608 lib/Target/RISCV/RISCV.h
lib/Target/RISCV/RISCVAsmPrinter.cpp
lib/Target/RISCV/RISCVISelLowering.h
lib/Target/RISCV/RISCVISelLowering.cpp
lib/Target/RISCV/RISCVMCInstLower.cpptest/CodeGen/RISCV/mem.ll
test/CodeGen/RISCV/wide-mem.ll
|
FYI, the diff here is confusing and part of what made me request you split this. If you'd extracted the helper function as a separate NFC change without review (which would have been fine), the reduced diff would have been a lot more obvious.