- Address comments
I've read this through again and it looks good. If no one else has any issues, then LGTM.
@jmorse Thanks for the comments!
Build result: fail - 59344 tests passed, 2 failed and 812 were skipped.
failed: LLVM.tools/llvm-ar/mri-utf8.test failed: LLVM.tools/llvm-objdump/X86/disassemble-functions.test
Build result: fail - 59544 tests passed, 1 failed and 805 were skipped.
Just a rebase.
Update to m_WithOverflowInst for now
Temporarily comment out call support as free probe, everything else passes validation but a call may have some stack effect I don't handle (yet).
An alternative approach is to generalize else if (EnsureSymtab) for non --add-symbol= operations as well.
Yeah, sorry. I've also been adding selection from arm's qadd/qsub, but I think that will be OK to standard types. Let me try and update a few things and then rebase this to see how it still does. Like you said, unsigned might still look OK.