gandhi21299 marked 3 inline comments as done. gandhi21299 added inline comments.
================ Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:12195 + if (!fpModeMatchesGlobalFPAtomicMode(RMW)) + return reportUnsafeHWInst(RMW, AtomicExpansionKind::None); ---------------- rampitec wrote: > rampitec wrote: > > This is wrong. Condition is inverted and essentially tests should fail. > > Make sure you can pass testing before posting a diff. > Unresolved. Remarks are produced if `fpModeMatchesGlobalFPAtomicMode(RMW) == false` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108150/new/ https://reviews.llvm.org/D108150 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits