Wed, May 5
increase version in test to preseve "simulator" + use removeSimulator() approach
Tue, May 4
add more tests
added some tests
Fri, Apr 23
LGTM - good catch! Is there not any sanitizers build that would've caught this?
Thu, Apr 22
Hm, I'd prefer not to have to see spurious clang-format messages.
It also means that I can't just clang-format -i the entire directory any more. I know there's clang-format-diff, but sometimes cleaning up the whole directory is what's needed...
How about introducing a multilineError function that takes multiple string parameters instead and adds the appropriate \n and >>> bits?
Mon, Apr 12
Apr 8 2021
Apr 7 2021
I actually snuck in a clang-format cleanup yesterday :) https://github.com/llvm/llvm-project/commit/174deb0539ee4af7d20bfead9f73055782e878e3 (so just update the commit title I guess)
removed unnecessary braces
Apr 6 2021
Apr 3 2021
- stop excluding debug sessions from reloc-parsing (cos we need the info)
- added some test
Apr 2 2021
Apr 1 2021
Mar 30 2021
Looks fairly straightforward to me
Mar 29 2021
Mar 25 2021
This is for https://bugs.llvm.org/show_bug.cgi?id=49282 right? For testing you could create a file with an undefined symbol. See e.g. test/ELF/undef.s.
updated diff to include the whole patch
@int3: not sure how to write a good test for this ?
The detailed graph looks nice! Thanks!
Mar 24 2021
Mar 23 2021
Updated path-expectations to accept both backslash and forwardslash.
@smeenai Thanks for the fix!
I've sent D99210
Updated diff. rebase.
Mar 22 2021
Updated diff: explicitly specified "signed char" to avoid ambiguity
Thanks for the revert! May be this patch is cursed 😕
- fixed test
- fixed build failure at https://lab.llvm.org/buildbot/#/builders/68/builds/9121/steps/4/logs/stdio
bot still unhappy ... reverting again: https://reviews.llvm.org/rG9670d2e4af4c996098089e31b03ca138bc8d27e9
moved inline functions to header