Removing lib/MC/MCObjectDisassembler (and more?)

Seems to be largely untested... anyone using this? (cc'ing Kevin as
he's the most likely candidate). At the least for MCAnalysis this
appears to be unused, any ideas on what else can be deleted?

I'll remove just this file unless there are objections tomorrow (or
earlier if I get an ack from Kevin about it).

-eric

Eric, what file are your referring to? The path lib/MC/MCObjectDisassembler does not end in a .cpp or .h and is not a directory .

Just a bit more info and I can get back to you if I’m using what your asking about.

Kev

Oops sorry. lib/MC/MCAnalysis/MCObjectDisassembler.CPP

Thanks.

-eric

Nope not using that. You might ask Jim as I think one of our interns added that with his work one summer.

Kev

I may not be reading this email correctly - isn't the ability to disassemble objects an advertised feature? I guess if you're removing that capability - make sure to update the website as well..

Is FreeBSD using this as part of their GNU toolchain replacement?

Seems to be largely untested... anyone using this? (cc'ing Kevin as
he's the most likely candidate). At the least for MCAnalysis this
appears to be unused, any ideas on what else can be deleted?

I'll remove just this file unless there are objections tomorrow (or
earlier if I get an ack from Kevin about it).

I may not be reading this email correctly - isn't the ability to disassemble
objects an advertised feature? I guess if you're removing that capability -
make sure to update the website as well..

You aren't reading this email correctly. There are other interfaces to
disassemble.

-eric

I don’t mind removing it as long as nothing is using it. If we, or anyone else, starts up that sort of work again, it’s easy enough to resurrect the code. It’s not helping anyone sitting in tree unused and untested.

-Jim

OK, removed thusly:

dzur:~/sources/llvm> git svn dcommit
Committing to https://llvm.org/svn/llvm-project/llvm/trunk ...
D lib/MC/MCAnalysis/MCObjectDisassembler.cpp
Committed r214437

Someone should probably figure out at some point what else can be removed here.

-eric

We’re (lightly) using it, but are happy to switch to something else. Is there a recommended replacement?

We're (lightly) using it, but are happy to switch to something else. Is
there a recommended replacement?

Not really, happy if you're using it to keep it. It'd be nice if it
the various aspects were tested :slight_smile:

-eric