[PATCH] D43814: [x86][CET] Introduce _get_ssp, _inc_ssp intrinsics
Craig Topper via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 2 09:55:06 PST 2018
craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: lib/Headers/cetintrin.h:45
+static __inline__ void __DEFAULT_FN_ATTRS _inc_ssp(unsigned int __a) {
+ __builtin_ia32_incsspq(__a);
+}
----------------
GBuella wrote:
> craig.topper wrote:
> > Where is the zeroing behavior for older CPUs coming from? This implementation looks identical to _incsspq?
> Do we need some zeroing behaviour for _inc_ssp?
> I know we need it for _get_ssp.
Nevermind. I mixed up the instructions, and didn't realize that we already modeled rdsspq/rdsspd as taking a value as input. I assumed we would have to do more work. Should have looked more closely. Sorry.
https://reviews.llvm.org/D43814
More information about the cfe-commits
mailing list