Historically, AMD internal sp3 assembler has flat_store* addr, data format. To match existing code and to enable reuse, change LLVM definitions to match.
Also update MC and CodeGen tests.
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
test/CodeGen/AMDGPU/sint_to_fp.ll | ||
---|---|---|
106 | You should leave these trailing commas. These are to prevent doing something like matching v1 and succeeding on v10. |
test/CodeGen/AMDGPU/sint_to_fp.ll | ||
---|---|---|
106 | This [[RESULT]] is the last operand of flat_store_dword after the change. I did not find any better way to match it, any suggestions? |
test/CodeGen/AMDGPU/sint_to_fp.ll | ||
---|---|---|
106 | You can use {{$}} to match the end of line |
test/CodeGen/AMDGPU/sint_to_fp.ll | ||
---|---|---|
106 | Then it won't match for buffer, because it's not last operand. |
test/CodeGen/AMDGPU/sint_to_fp.ll | ||
---|---|---|
106 | We can use a different lines with a different check prefix for SI/VI, although it's probably not worth it for this. This is good enough |
test/CodeGen/AMDGPU/sint_to_fp.ll | ||
---|---|---|
106 | If this is good enough, could you ok the review? |
You should leave these trailing commas. These are to prevent doing something like matching v1 and succeeding on v10.