[PATCH] Add iterator and iterator range to PredIteratorCache
David Blaikie
dblaikie at gmail.com
Tue Apr 21 13:53:49 PDT 2015
Looks reasonable.
Optional: Rather than having range access as a free function and begin/end as member functions, I'd be inclined to put them at the same level, or possibly even go without begin/end entirely and just have:
PredCache.predecessors(BB);
(or s/predecessors/cached_predecessors/ - though of course both names are somewhat redundant with the name of the object... - maybe it should just be PredCache.get(BB); or something? )
http://reviews.llvm.org/D9169
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list