- User Since
- Oct 5 2020, 4:48 PM (31 w, 3 d)
Feb 24 2021
@lenary Thanks for looking at this. Indeed, those uses of %i where it should be %u were oversights.
Feb 19 2021
Ace, thank you.
Feb 18 2021
I had not intended to create this new diff - abandoning.
@lenary Thanks for the review! I have fixed the compiler warning that you spotted. I hope I did the right thing when I uploaded a revised patch (this is my first time working with the differential/arcanist workflow).
Fixed a compiler warning as spotted by @lenary
Oct 6 2020
Hi @anemet and @lenary - I am keen to contribute these tests that improve on the code coverage achieved by the existing test suite. It's my first LLVM-related patch so I wasn't sure who the right folks to ask for reviews were, but I saw that you'd each done some recent reviews in the test suite so I wondered whether you might be suitable to review this, or to suggest other reviewers.