piyushtripathi9424 commented on issue #17992: URL: https://github.com/apache/nuttx/issues/17992#issuecomment-3782704685
Thanks for the update, @Y334275 , and for testing the change from INT32_C(x) x ## ll to INT32_C(x) x ## l. That matches the expectation that INT32_C should have the same underlying type as int32_t on xtensa, so it makes sense that it fixes the template mismatch in math::constrain. I will go through the discussion in #15755 and PR #16022 as suggested to make sure the change is consistent with previous decisions, and then I’d be happy to help with or review a PR that adjusts the macro accordingly. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
