davide added a comment.
In https://reviews.llvm.org/D28799#648062, @ioeric wrote:
> @davide I think this change makes sense. I'll accept this to unbreak our
> internal build. Let us know if you have any concern.
Yes, makes sense. Thanks.
Repository:
rL LLVM
https://reviews.llvm.org/D28799
This revision was automatically updated to reflect the committed changes.
Closed by commit rL29: [llvm-objdump tests] Copy the inputs of tests closer
to tests. (authored by krasimir).
Changed prior to commit:
https://reviews.llvm.org/D28799?vs=84653&id=84667#toc
Repository:
rL LLVM
http
ioeric accepted this revision.
ioeric added a comment.
This revision is now accepted and ready to land.
@davide I think this change makes sense. I'll accept this to unbreak our
internal build. Let us know if you have any concern.
https://reviews.llvm.org/D28799
__
krasimir created this revision.
Tests under tools/llvm-objdump should not use inputs from Object. Copied the
required inputs and aligned the new tests to be more consistent with the
existing
tests in this respect.
https://reviews.llvm.org/D28799
Files:
test/tools/llvm-objdump/X86/Inputs/open