[PATCH] D102347: [AMDGPU] Only allow global fp atomics with unsafe option

[Tony Tye via Phabricator via llvm-commits]

t-tye added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:12147-12153
     // DS FP atomics do repect the denormal mode, but the rounding mode is fixed
     // to round-to-nearest-even.
     // The only exception is DS_ADD_F64 which never flushes regardless of mode.
     if (AS == AMDGPUAS::LOCAL_ADDRESS && Subtarget->hasLDSFPAtomics()) {
       return (Ty->isDoubleTy() && !fpModeMatchesGlobalFPAtomicMode(RMW)) ?
         AtomicExpansionKind::CmpXChg : AtomicExpansionKind::None;
     }
----------------
Should the amdgpu-unsafe-fp-atomics also ally here too? If not should the attribute be named to reflect it only allies to global fp atomics?


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

https://reviews.llvm.org/D102347