I don't feel strongly about it (and don't know the codebase much) but there's similar code here: lib/Target/ARM/MCTargetDesc/ARMMCTargetDesc.cpp
I'll defer to those who know LLVM better than I do for this, and it otherwise looks good to me. On Mon, May 13, 2013 at 2:58 PM, Rafael Espíndola < [email protected]> wrote: > On 13 May 2013 17:55, Renato Golin <[email protected]> wrote: > > Looks better. Have you ran check all? > > Yes. > > Cheers, > Rafael >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
