Commit 58bf9ac
authored
[NVPTX] Remove unsupported 'seq_cst' test (llvm#156088)
The NVPTX backend does not currently support correctly lowering
`atomicrmw` with `seq_cst` of any size. Remove a test which erroneously
hit this case and add logic to appropriately error out when this
ordering is encountered, instead of emitting invalid PTX.
In the long term, we should use `fence.sc`, similar to other atomic
operations, to support this ordering.1 parent 01c0a84 commit 58bf9ac
File tree
2 files changed
+4
-3
lines changed- llvm
- lib/Target/NVPTX/MCTargetDesc
- test/CodeGen/NVPTX
2 files changed
+4
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
290 | 290 | | |
291 | 291 | | |
292 | 292 | | |
293 | | - | |
| 293 | + | |
| 294 | + | |
294 | 295 | | |
295 | 296 | | |
296 | 297 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1024 | 1024 | | |
1025 | 1025 | | |
1026 | 1026 | | |
1027 | | - | |
| 1027 | + | |
1028 | 1028 | | |
1029 | 1029 | | |
1030 | 1030 | | |
1031 | | - | |
| 1031 | + | |
1032 | 1032 | | |
1033 | 1033 | | |
0 commit comments