NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit85c9dda

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 parent9208c6f commit85c9ddaCopy full SHA for 85c9dda

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
`@@ -3102,10 +3102,11 @@ timetz_zone(PG_FUNCTION_ARGS)`
`3102`	`3102`	`result= (TimeTzADT*)palloc(sizeof(TimeTzADT));`
`3103`	`3103`
`3104`	`3104`	`result->time=t->time+ (t->zone-tz)*USECS_PER_SEC;`
	`3105`	`+/* C99 modulo has the wrong sign convention for negative input */`
`3105`	`3106`	`while (result->time<INT64CONST(0))`
`3106`	`3107`	`result->time+=USECS_PER_DAY;`
`3107`		`-while (result->time >=USECS_PER_DAY)`
`3108`		`-result->time-=USECS_PER_DAY;`
	`3108`	`+if (result->time >=USECS_PER_DAY)`
	`3109`	`+result->time%=USECS_PER_DAY;`
`3109`	`3110`
`3110`	`3111`	`result->zone=tz;`
`3111`	`3112`
`@@ -3135,10 +3136,11 @@ timetz_izone(PG_FUNCTION_ARGS)`
`3135`	`3136`	`result= (TimeTzADT*)palloc(sizeof(TimeTzADT));`
`3136`	`3137`
`3137`	`3138`	`result->time=time->time+ (time->zone-tz)*USECS_PER_SEC;`
	`3139`	`+/* C99 modulo has the wrong sign convention for negative input */`
`3138`	`3140`	`while (result->time<INT64CONST(0))`
`3139`	`3141`	`result->time+=USECS_PER_DAY;`
`3140`		`-while (result->time >=USECS_PER_DAY)`
`3141`		`-result->time-=USECS_PER_DAY;`
	`3142`	`+if (result->time >=USECS_PER_DAY)`
	`3143`	`+result->time%=USECS_PER_DAY;`
`3142`	`3144`
`3143`	`3145`	`result->zone=tz;`
`3144`	`3146`

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!

Commit85c9dda

File tree

1 file changed

1 file changed

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

0 commit comments