[Lldb-commits] [PATCH] D56293: Use the minidump exception record if present
Zachary Turner via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Jan 3 15:08:43 PST 2019
zturner requested changes to this revision.
zturner added a comment.
This revision now requires changes to proceed.
I don't think we can check in an executable file, we should try to compile it on the spot. We have 1-2 existing unit tests that check in an exe and we occasionally get reports that peoples' virus scanners flag them as trojans, even though they obviously aren't. In any case, I've been meaning to remove those tests, so I think we should set a precedent that executable binaries are never checked in.
Is there something about this executable that makes it impractical to compile on the fly using the `%build` substitution?
CHANGES SINCE LAST ACTION
More information about the lldb-commits