[llvm-commits] [llvm] r109220 - /llvm/trunk/include/llvm/Use.h
ggreif at gmail.com
Fri Jul 23 02:08:33 PDT 2010
Sorry for the damage, I'll take care of llvm-gcc this week.
Will write up a small blurb for the release notes too.
Btw. why is the buildbot site down?
On Jul 23, 10:53 am, Eric Christopher <echri... at apple.com> wrote:
> On Jul 23, 2010, at 1:46 AM, Eric Christopher wrote:
> > On Jul 23, 2010, at 1:36 AM, Gabor Greif wrote:
> >> Author: ggreif
> >> Date: Fri Jul 23 03:36:07 2010
> >> New Revision: 109220
> >> URL:http://llvm.org/viewvc/llvm-project?rev=109220&view=rev
> >> Log:
> >> Simplifying use_iterators by dereferencing
> >> is not a good idea. The codebase does not depend
> >> in this any more, and it may introduce hidden
> >> runtime cost. If you get compile errors, please
> >> dereference your iterator before passing to cast<>
> >> (and friends).
> >> Also: please consider caching the result of
> >> operator* and reusing that instead of dereferencing
> >> many times.
> > Please revert this, you broke llvm-gcc and clang.
> I"ve gone ahead and reverted it.
> llvm-commits mailing list
> llvm-comm... at cs.uiuc.eduhttp://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
More information about the llvm-commits