Re: LIPO based on LTO IR streaming (issue 5746044)

2012-03-16 Thread davidxl
ok for google-46 after the minor changes below. Make sure default lipo testing is well covered too. David http://codereview.appspot.com/5746044/diff/11001/cgraph.c File cgraph.c (right): http://codereview.appspot.com/5746044/diff/11001/cgraph.c#newcode2887 cgraph.c:2887:

Re: LIPO based on LTO IR streaming (issue 5746044)

2012-03-16 Thread stevenb . gcc
One spelling nit. May also want to explain the 'ripa' term more clearly somewhere. http://codereview.appspot.com/5746044/diff/11001/ipa.c File ipa.c (right): http://codereview.appspot.com/5746044/diff/11001/ipa.c#newcode811 ipa.c:811: (2) the VNODE is neede, and s/neede/needed/