[PATCH] D159115: [clang-repl] Adapt to the recent dylib-related changes in ORC.
Tulio Magno Quites Machado Filho via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Sep 8 10:09:15 PDT 2023
tuliom added a comment.
In D159115#4641834 <https://reviews.llvm.org/D159115#4641834>, @mgorny wrote:
> Is that actually a regression, or merely the test wasn't checking it before?
I can't tell because the test didn't exist before.
> My educated guess would be that the code in the test redefines `printf` without the header mangling, so it finds and compares the "non-IEEE" symbol against the "IEEE" symbol that's exposed to the test case via the headers.
I agree with you.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D159115/new/
https://reviews.llvm.org/D159115
More information about the cfe-commits
mailing list