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: && !(L_IPO_STREAM_IN_LTO_P &&
cgraph_is_auxiliary(node->decl)))
Missing space

http://codereview.appspot.com/5746044/diff/11001/coverage.c
File coverage.c (right):

http://codereview.appspot.com/5746044/diff/11001/coverage.c#newcode396
coverage.c:396: warning (0, "error in opening %s, exiting",
0 -> UNKNOWN_LOCATION

http://codereview.appspot.com/5746044/diff/11001/coverage.c#newcode724
coverage.c:724: module_infos, num_in_fnames);
Identation

http://codereview.appspot.com/5746044/diff/11001/ipa.c
File ipa.c (right):

http://codereview.appspot.com/5746044/diff/11001/ipa.c#newcode678
ipa.c:678: Return true will make this symbol externally visible.
Returning

http://codereview.appspot.com/5746044/diff/11001/ipa.c#newcode679
ipa.c:679: Refurn false will continue current visibiliity logic.  */
Returning

http://codereview.appspot.com/5746044/

Reply via email to