[PATCH] Add support for part-word atomics for PPC

hfinkel at anl.gov hfinkel at anl.gov
Fri Mar 6 06:44:14 PST 2015


REPOSITORY
  rL LLVM

================
Comment at: lib/Target/PowerPC/PPCInstrInfo.td:1469
@@ +1468,3 @@
+                   "lharx $rD, $src", IIC_LdStLWARX,
+                   [(set i32:$rD, (PPClarx xoaddr:$src))]>,
+                   Requires<[HasPartwordAtomics]>;
----------------
This pattern is identical to the pattern for LBARX above; can they both really be selected?

(and the same as the LWARX pattern below)

Maybe you need PPClarx to carry a type node operand, like the extload nodes?


================
Comment at: lib/Target/PowerPC/PPCInstrInfo.td:1485
@@ +1484,3 @@
+                   "lharx $rD, $src, 1", IIC_LdStLWARX,
+                   [(set i32:$rD, (PPClarxl xoaddr:$src))]>, isDOT,
+                   Requires<[HasPartwordAtomics]>;
----------------
Duplicate pattern here too.

http://reviews.llvm.org/D8090

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list