[llvm-commits] [llvm] r150775 - in /llvm/trunk/lib/Target/Mips: ./ AsmParser/ InstPrinter/ MCTargetDesc/ TargetInfo/
Chris Lattner
clattner at apple.com
Fri Feb 17 00:57:27 PST 2012
On Feb 16, 2012, at 8:24 PM, Jia Liu wrote:
> On Fri, Feb 17, 2012 at 12:08 PM, Chandler Carruth <chandlerc at google.com> wrote:
>> On Thu, Feb 16, 2012 at 8:02 PM, Jia Liu <proljc at gmail.com> wrote:
>>>
>>> Oh! Sorry. I thought it should be because I saw it in some other .cpp
>>> files.
>>
>>
>> I think those files are the exceptions, and not the rules. I think the goal
>> is to have the minimum number of these required.
>
> Yes, my understanding was wrong.
Yep, please remove these from .cpp files when you see them. Not a huge problem, but I'd appreciate it if you could remove them from the mips .cpp files too. Thanks!
-Chris
More information about the llvm-commits
mailing list