[llvm-commits] PATCH: clean up usage of the RelaxAll flag in MC

Rafael EspĂ­ndola rafael.espindola at gmail.com
Tue Dec 11 08:58:22 PST 2012


>> assert(!getRelaxAll() && "MCInstFragment created unnecessarily");
>>
>
> I like asserts, but I would prefer to put this one higher in the call
> tree. Say in relaxInstruction, or even in layoutSectionOnce when it
> sees a FT_Inst fragment. What do you say?

LGTM!

> Eli

Cheers,
Rafael



More information about the llvm-commits mailing list