Skip to content

UndefinedBehaviorSanitizer: signed integer overflow issue#793

Open
sam-svutkoor wants to merge 1 commit intoqualcomm:mainfrom
sam-svutkoor:sanity-fix
Open

UndefinedBehaviorSanitizer: signed integer overflow issue#793
sam-svutkoor wants to merge 1 commit intoqualcomm:mainfrom
sam-svutkoor:sanity-fix

Conversation

@sam-svutkoor
Copy link

eld/lib/Target/RISCV/RISCVHelper.h:243:18: runtime error: signed integer overflow: 2147483628 + 2048 cannot be represented in type 'int'
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior RISCVHelper.h:243:18
this commit fixes the build failure for picolibc riscv64-bm sanity
Removes signed overflow UBSan complaining

@quic-seaswara
Copy link
Contributor

pls sign the commit. thanks for the patch

@sam-svutkoor sam-svutkoor force-pushed the sanity-fix branch 4 times, most recently from 4b5b810 to ec5089f Compare February 10, 2026 00:54
Copy link
Contributor

@quic-seaswara quic-seaswara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove change id from the commit message

@sam-svutkoor sam-svutkoor force-pushed the sanity-fix branch 2 times, most recently from e4264c1 to 81dc581 Compare February 11, 2026 15:37
Copy link
Member

@lenary lenary left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

I think this is right, but it's not very neat. I think anything that would make this a neater change would be quite involved, though.

runtime error: signed integer overflow: 2147483628 + 2048 cannot be represented in type 'int' fix

Signed-off-by: svutkoor <svutkoor@qti.qualcomm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants