[PATCH] D102371: [AMDGPU] Refactor shouldExpandAtomicRMWInIR(). NFC.

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 12 14:23:55 PDT 2021


arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:12113
 
-    if (!Ty->isFloatTy() && (!Subtarget->hasGFX90AInsts() || !Ty->isDoubleTy()))
+    if (!Ty->isFloatTy() && !(Subtarget->hasGFX90AInsts() && Ty->isDoubleTy()))
       return AtomicExpansionKind::CmpXChg;
----------------
I think the previous demorganed version is easier


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

https://reviews.llvm.org/D102371



More information about the llvm-commits mailing list