NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit1ad3a7b

committed

Dodge a compiler bug affecting timetz_zone/timetz_izone.

This avoids a compiler bug occurring in AIX's xlc, even in prettylate-model revisions. Buildfarm testing has now confirmed thatonly 64-bit xlc is affected. Although we are contemplatingdropping support for xlc in v17, it's still supported in theback branches, so we need this fix.Back-patch of code changes from HEAD commit19fa977.(The test cases were already back-patched, in4a427b8 et al.)Discussion:https://postgr.es/m/CA+hUKGK=DOC+hE-62FKfZy=Ybt5uLkrg3zCZD-jFykM-iPn8yw@mail.gmail.com

1 parent1856a0e commit1ad3a7bCopy full SHA for 1ad3a7b

File tree

1 file changed

-4

lines changed

src/backend/utils/adt
- date.c

1 file changed

-4

lines changed

`‎src/backend/utils/adt/date.c‎`

Lines changed: 6 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -3083,10 +3083,11 @@ timetz_zone(PG_FUNCTION_ARGS)`
`3083`	`3083`	`result= (TimeTzADT*)palloc(sizeof(TimeTzADT));`
`3084`	`3084`
`3085`	`3085`	`result->time=t->time+ (t->zone-tz)*USECS_PER_SEC;`
	`3086`	`+/* C99 modulo has the wrong sign convention for negative input */`
`3086`	`3087`	`while (result->time<INT64CONST(0))`
`3087`	`3088`	`result->time+=USECS_PER_DAY;`
`3088`		`-while (result->time >=USECS_PER_DAY)`
`3089`		`-result->time-=USECS_PER_DAY;`
	`3089`	`+if (result->time >=USECS_PER_DAY)`
	`3090`	`+result->time%=USECS_PER_DAY;`
`3090`	`3091`
`3091`	`3092`	`result->zone=tz;`
`3092`	`3093`
`@@ -3116,10 +3117,11 @@ timetz_izone(PG_FUNCTION_ARGS)`
`3116`	`3117`	`result= (TimeTzADT*)palloc(sizeof(TimeTzADT));`
`3117`	`3118`
`3118`	`3119`	`result->time=time->time+ (time->zone-tz)*USECS_PER_SEC;`
	`3120`	`+/* C99 modulo has the wrong sign convention for negative input */`
`3119`	`3121`	`while (result->time<INT64CONST(0))`
`3120`	`3122`	`result->time+=USECS_PER_DAY;`
`3121`		`-while (result->time >=USECS_PER_DAY)`
`3122`		`-result->time-=USECS_PER_DAY;`
	`3123`	`+if (result->time >=USECS_PER_DAY)`
	`3124`	`+result->time%=USECS_PER_DAY;`
`3123`	`3125`
`3124`	`3126`	`result->zone=tz;`
`3125`	`3127`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit1ad3a7b

File tree

1 file changed

1 file changed

`‎src/backend/utils/adt/date.c‎`

0 commit comments