[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