diff options
author | Greg Clayton <gclayton@apple.com> | 2016-06-24 23:48:00 +0000 |
---|---|---|
committer | Greg Clayton <gclayton@apple.com> | 2016-06-24 23:48:00 +0000 |
commit | d7f71add86c892ced324fc26cb0945911f3e26a5 (patch) | |
tree | 9ed27eb7686df44798786a1e5b80a0d9ec5c2e7f /lldb/packages/Python/lldbsuite/test/lang/cpp/template/main.cpp | |
parent | 4d32eb6939d77f8059683c008bb807167110e930 (diff) | |
download | bcm5719-llvm-d7f71add86c892ced324fc26cb0945911f3e26a5.tar.gz bcm5719-llvm-d7f71add86c892ced324fc26cb0945911f3e26a5.zip |
Made templates that have Enumeration values as arguments work correctly.
We were checking for integer types only before this. So I added the ability for CompilerType objects to check for integer and enum types.
Then I searched for places that were using the CompilerType::IsIntegerType(...) function. Many of these places also wanted to be checking for enumeration types as well, so I have fixed those places. These are in the ABI plug-ins where we are figuring out which arguments would go in where in regisers/stack when making a function call, or determining where the return value would live. The real fix for this is to use clang to compiler a CGFunctionInfo and then modify the code to be able to take the IR and a calling convention and have the backend answer the questions correctly for us so we don't need to create a really bad copy of the ABI in each plug-in, but that is beyond the scope of this bug fix.
Also added a test case to ensure this doesn't regress in the future.
llvm-svn: 273750
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/lang/cpp/template/main.cpp')
-rw-r--r-- | lldb/packages/Python/lldbsuite/test/lang/cpp/template/main.cpp | 49 |
1 files changed, 48 insertions, 1 deletions
diff --git a/lldb/packages/Python/lldbsuite/test/lang/cpp/template/main.cpp b/lldb/packages/Python/lldbsuite/test/lang/cpp/template/main.cpp index 10a4bec62e9..9c33a642091 100644 --- a/lldb/packages/Python/lldbsuite/test/lang/cpp/template/main.cpp +++ b/lldb/packages/Python/lldbsuite/test/lang/cpp/template/main.cpp @@ -17,9 +17,56 @@ public: } }; +//---------------------------------------------------------------------- +// Define a template class that we can specialize with an enumeration +//---------------------------------------------------------------------- +enum class EnumType +{ + Member, + Subclass +}; + +template <EnumType Arg> class EnumTemplate; + +//---------------------------------------------------------------------- +// Specialization for use when "Arg" is "EnumType::Member" +//---------------------------------------------------------------------- +template <> +class EnumTemplate<EnumType::Member> +{ +public: + EnumTemplate(int m) : + m_member(m) + { + } + + int getMember() const + { + return m_member; + } + +protected: + int m_member; +}; + +//---------------------------------------------------------------------- +// Specialization for use when "Arg" is "EnumType::Subclass" +//---------------------------------------------------------------------- +template <> +class EnumTemplate<EnumType::Subclass> : + public EnumTemplate<EnumType::Member> +{ +public: + EnumTemplate(int m) : EnumTemplate<EnumType::Member>(m) + { + } +}; + int main(int argc, char **argv) { TestObj<1> testpos; TestObj<-1> testneg; - return testpos.getArg() - testneg.getArg(); // Breakpoint 1 + EnumTemplate<EnumType::Member> member(123); + EnumTemplate<EnumType::Subclass> subclass(123*2); + return testpos.getArg() - testneg.getArg() + member.getMember()*2 - subclass.getMember(); // Breakpoint 1 } |