Also fix v_mac.ll not testing right thing for fneg
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
Apart from some minor comments, LGTM.
test/CodeGen/AMDGPU/fmul.ll | ||
---|---|---|
1 | Isn't it better to be explicit about -mcpu? Not that I care strongly either way... | |
test/CodeGen/AMDGPU/llvm.SI.sendmsg-m0.ll | ||
17–19 | Now #1 is missing in the sendmsg declare, isn't it? | |
test/CodeGen/AMDGPU/sgpr-copy.ll | ||
398–400 | Duplicate #1 and #3. |
Comment Actions
r275129
test/CodeGen/AMDGPU/fmul.ll | ||
---|---|---|
1 | It doesn't really matter, the default is just tahiti. I don't think most tests specify for SI |
Isn't it better to be explicit about -mcpu? Not that I care strongly either way...