appujee
appujee
> Thanks for the recent update! Only a further suggestion regarding tail-folding style handling and moving that to a separate NFC PR in preparation for this one Please +1 if...
@alexey-bataev @fhahn As Ayal has not responded to the patch in weeks (https://github.com/llvm/llvm-project/pull/76172#issuecomment-1942735278) even after pinging, should we follow up in a post commit review if and when Ayal has...
ping. any update on when this will land? cc: @Maratyszcza
https://github.com/torvalds/linux/commit/16252e018a30486eedcfec81fc313445cac25bea maybe helpful in revising this patch.
Can't repro on clang upstream https://godbolt.org/z/d9bGnhhv6 so the fix is likely to be in upstream.
The bug is in SimplifyCFGPass Before ```llvm define dso_local noundef i32 @main() #0 { %1 = alloca [2 x ptr], align 16 store ptr blockaddress(@main, %7), ptr %1, align 16,...
https://r.android.com/3211575
by the way, the patch i have uploaded 'hides' the issue. the real issue is still in llvm trunk. It could take a while to land a fix but I'm...
So the change in https://r.android.com/3211575 'updates the CFG' in a way that the later (buggy) pass can't see the problematic pattern anymore. Most likely the issue can't be reproduced in...
I believe https://github.com/llvm/llvm-project/commit/fc6bdb8549842613da51b9d570b29e27cc709f69 is the patch causing the bug. I'm preparing a fix but reverting this should also work.