[PATCH] D71128: [NVPTX][FIX] Expand atomics we cannot handle natively in the ISA

Fri Dec 6 13:39:10 PST 2019

tra added a comment.

  > Command Output (stderr):
  > --
  > /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang8/llvm/test/CodeGen/NVPTX/atomics-sm60.ll:6:10: error: CHECK: expected string not found in input
  > ; CHECK: atom.add.f64
  >          ^
  > <stdin>:1:1: note: scanning from here
  > //
  > ^
  > <stdin>:34:2: note: possible intended match here
  >  atom.cas.b64 %rd3, [%r1], %rd2, %rd1;
  >  ^

This appears to be a regression. We do have fp32/fp64 atomic adds in NVPTX. Replacing them with add+CAS is suboptimal.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71128/new/

https://reviews.llvm.org/D71128