OK, I'll go the other way.
On Tue, Apr 9, 2013 at 4:54 PM, John McCall <[email protected]> wrote: > On Apr 8, 2013, at 9:09 PM, Stephen Lin <[email protected]> wrote: >> CodeGenModule::getTarget() is implemented as a trivial call to >> CodeGenModule::getContext().getTargetInfo(), however, the two are >> called interchangeably from other functions without any apparent >> semantic distinction between the two. >> >> Since the latter usage is more prevalent and is more fundamental >> anyway, I've eliminated the former and converted all usages to the >> latter. > > I'd go the other way, actually: it's shorter, and we can cache it easily > if we decide it's important. > >> Also, I've removed CodeGenTypes::getTarget(), since it wasn't >> being used. > > This seems fine. > > John. _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
