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

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 9 07:15:27 PST 2020


arsenm added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/TargetLowering.h:1856-1871
+
+  /// See shouldTransformToIntegerOperation(Instruction *)
+  virtual bool shouldTransformToIntegerOperation(LoadInst *LI) const {
+    return shouldTransformToIntegerOperation(cast<Instruction>(LI));
+  }
+
+  /// See shouldTransformToIntegerOperation(Instruction *)
----------------
Do we really need 4 of these when just the one for Instruction will work


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71128





More information about the llvm-commits mailing list