Hello Anton, "arm" is a typo. My mistake. I will fix this. Sorry for inconvenience.
Test case: test/CodeGenCXX/member-function-pointers.cpp It is extended with new RUN line for MIPS. New representation of pointer to member function is the same as for ARM. This patch fixes an issue with MultiSource/Benchmarks/Bullet test when compiled for microMIPS. Review was done here: http://reviews.llvm.org/D7148 Regards, Zoran ________________________________________ From: Anton Korobeynikov [[email protected]] Sent: Monday, February 23, 2015 4:18 PM To: Zoran Jovanovic Cc: cfe-commits Subject: Re: r229680 - Change representation of member function pointers for MIPS targets > --- cfe/trunk/lib/Basic/TargetInfo.cpp (original) > +++ cfe/trunk/lib/Basic/TargetInfo.cpp Wed Feb 18 09:21:35 2015 > @@ -655,6 +655,7 @@ bool TargetCXXABI::tryParse(llvm::String > .Case("ios", iOS) > .Case("itanium", GenericItanium) > .Case("microsoft", Microsoft) > + .Case("arm", GenericMIPS) > .Default(unknown); > if (kind == unknown) return false; Why "arm" here? Did anyone reviewed this patch? Where are the tests? -- With best regards, Anton Korobeynikov Faculty of Mathematics and Mechanics, Saint Petersburg State University _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
