6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2024-12-13 04:30:23 +00:00
Code Issues Projects Releases Wiki Activity

fix PR6589

Browse Source 
adjusted unittest

I have added some doxygen to OptionalOperandTraits,
so hopefully there will be no confusion in the future.

Incidentally OptionalOperandTraits is not used any more (IIUC),
but the obvious client would be BranchInstr, and I plan
to rearrange it that way.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@98624 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Gabor Greif 2010-03-16 10:59:48 +00:00
parent 5a7069a80f
commit 138acfe353
3 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
include/llvm/Instructions.h
View File

@ -1984,7 +1984,7 @@ public:
};
template <>
struct OperandTraits<ReturnInst> : public OptionalOperandTraits<> {
struct OperandTraits<ReturnInst> : public VariadicOperandTraits<> {
};
DEFINE_TRANSPARENT_OPERAND_ACCESSORS(ReturnInst, Value)

7
include/llvm/OperandTraits.h
View File

@ -20,7 +20,7 @@
namespace llvm {
//===----------------------------------------------------------------------===//
// FixedNumOperands Trait Class
// FixedNumOperand Trait Class
//===----------------------------------------------------------------------===//
/// FixedNumOperandTraits - determine the allocation regime of the Use array
@ -51,9 +51,12 @@ struct FixedNumOperandTraits {
};
//===----------------------------------------------------------------------===//
// OptionalOperands Trait Class
// OptionalOperand Trait Class
//===----------------------------------------------------------------------===//
/// OptionalOperandTraits - when the number of operands may change at runtime.
/// Naturally it may only decrease, because the allocations may not change.
template <unsigned ARITY = 1>
struct OptionalOperandTraits : public FixedNumOperandTraits<ARITY> {
static unsigned operands(const User *U) {

6
unittests/VMCore/InstructionsTest.cpp
View File

@ -14,12 +14,12 @@
namespace llvm {
namespace {
TEST(InstructionsTest, ReturnInst_0) {
TEST(InstructionsTest, ReturnInst) {
LLVMContext &C(getGlobalContext());
// reproduction recipe for PR6589
// test for PR6589
const ReturnInst* r0 = ReturnInst::Create(C);
EXPECT_NE(r0->op_begin(), r0->op_end());
EXPECT_EQ(r0->op_begin(), r0->op_end());
}
} // end anonymous namespace