From a5a674fa2c2447e9dd34c8c8f7b1e5756a32df4d Mon Sep 17 00:00:00 2001 From: "Frank Ch. Eigler" Date: Tue, 24 Nov 2009 15:46:41 -0500 Subject: PR10601: comment on loc2c's use of [u]intptr_t for temp values --- loc2c.c | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'loc2c.c') diff --git a/loc2c.c b/loc2c.c index d3ec55fa..a7473784 100644 --- a/loc2c.c +++ b/loc2c.c @@ -30,6 +30,13 @@ #define N_(x) x +/* NB: PR10601 may make one suspect that intptr_t and uintptr_t aren't + right, for example on a 64-bit kernel targeting a 32-bit userspace + process. At least these types are always at least as wide as + userspace (since 64-bit userspace doesn't run on a 32-bit kernel). + So as long as deref() and {fetch,store}_register() widen/narrow + their underlying values to these, there should be no problem. */ + #define STACK_TYPE "intptr_t" /* Must be the signed type. */ #define UTYPE "uintptr_t" /* Must be the unsigned type. */ #define SFORMAT "%" PRId64 "L" -- cgit