- User Since
- Apr 4 2021, 4:02 PM (95 w, 5 d)
Dec 11 2022
Fixed typo. Sorry!
Dec 9 2022
Add (Initializer && and assert to else branch per Tom's comments.
Dec 8 2022
Add assert at the beginning of blocks pointed out by Coverity
Dec 6 2022
Dec 5 2022
Perhaps the` assert` should be added after line 5926 above.
I think the checks for Initializer being non-null following the addition of an assert should be removed.
I don't see any after this point.
Move the assert to after the branches that handle the cases where Initializer may be null
Dec 2 2022
Add assert instead of multiple checks in if conditions.
Add more nullptr checks per Shafik's comments
Dec 1 2022
Yes, it does make sense.
I am guessing the answer is no since this was found using a static analysis tool but do you have a test case that triggers this failure?
Indeed, the answer is no.
Dec 15 2021
Dec 14 2021
Change CHECK-LABEL to STORE-LINE-LABEL
Rebasing patch to see if the test failure will go away.
Test failure seems unrelated to this patch. Unable to reproduce locally.
Dec 13 2021
Fix test format
Changing test's CHECK lines
Dec 10 2021
- Removed returnNullTBAA() method
- Add NO-TBAA check-prefix to the test
Dec 9 2021
Updated test per Jeroen's suggestion
Dec 8 2021
Add test case
Dec 7 2021
Aug 25 2021
Aug 23 2021
- Removed the useless assert on TC.
- Retained the change from cast_or_null to cast