[PATCH] D119802: [HWASan] remove replacement of DbgVariableIntrinsics.
Evgenii Stepanov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 15 11:15:32 PST 2022
eugenis added a comment.
In D119802#3321609 <https://reviews.llvm.org/D119802#3321609>, @fmayer wrote:
> In D119802#3321598 <https://reviews.llvm.org/D119802#3321598>, @fmayer wrote:
>
>> Oh hold on, maybe what we actually want (to fix this) is to replace `NewPtr` (the BitCast from `alignAndPadAlloca`) with `Info.AI`. Sorry for jumping the gun here, the change as is is correct in that it keeps current behaviour, but that is possibly not what we want.
>
> Nevermind, no. We actually want the cast, because otherwise we cast to the (wrong) type of the padded alloca; or at least that's what the test in asserting. Correct me if that's wrong.
Which test?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119802/new/
https://reviews.llvm.org/D119802
More information about the llvm-commits
mailing list