Skip to content

Conversation

@dpalermo
Copy link

@dpalermo dpalermo commented Nov 8, 2024

- Use actual function for memmove (address taken)

    - Use actual function for memmove (address taken)
@dpalermo
Copy link
Author

dpalermo commented Nov 8, 2024

Fixed needed after change in llvm#114301

@dpalermo
Copy link
Author

dpalermo commented Nov 8, 2024

Also interesting, but the pre-existing libc replacement memmove was missing a return value on the early exit case. Now fixed. Reported as a compilation error after no longer using the memmove builtin.

@dpalermo
Copy link
Author

dpalermo commented Nov 8, 2024

Raghu noticed the GenASiS target test was no longer finding -lFortranRuntimeHostDevice ...as the build of libFortranRuntimeHostDevice.a was failing.

Copy link

@raghavendhra raghavendhra left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this :)

@dpalermo dpalermo merged commit 27e3c3a into amd-trunk-dev Nov 8, 2024
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants