Branch: refs/heads/main
Revision: b45275f
Author: arezaii
Link: Fix: failing cron asan tests in next/primers by arezaii · Pull Request #19026 · chapel-lang/chapel · GitHub
Log Message:
Merge pull request #19026 from arezaii/primer-test-fix
Fix: failing cron asan tests in next/primers
This PR fixes an issue where testing with the
address sanitizer caused several next/primer
tests to fail.
We wrap a call to c_str() in astr() to prevent an
instance of use-after-scope that happened when
visiting a Conditional node during the AST conversion
process.
TESTING:
all with LLVM=none
- [x] linux testing with address sanitizer
- [x] mac testing with address sanitizer
- [x] paratest
Reviewed by @dlongnecke-cray, thanks!
Signed-off-by: arezaii ahmad.rezaii@hpe.com
Modified Files:
M compiler/passes/convert-uast.cpp
Compare: https://github.com/chapel-lang/chapel/compare/793fb4a5e558...b45275f323b1