Re: [google][4.6]Add new target builtin to check for amdfam15h processors (issue5535046)

2012-01-10 Thread Sriraman Tallam
On Tue, Jan 10, 2012 at 3:33 PM, Andrew Pinski pins...@gmail.com wrote: On Tue, Jan 10, 2012 at 3:31 PM, Diego Novillo dnovi...@google.com wrote: On 12-01-10 17:11 , Sriraman Tallam wrote: This patch adds a new target builtin, __builtin_cpu_is_amdfam15, to check for AMD Family 15h processors

Re: [patch i386][google][4.6]Add new target builtins to check for corei7 and amdfam10 (issue5495075)

2011-12-17 Thread Sriraman Tallam
Checked in. Thanks, -Sri. On Sat, Dec 17, 2011 at 12:21 AM, Xinliang David Li davi...@google.com wrote: ok for google branches. David On Fri, Dec 16, 2011 at 7:54 PM, Sriraman Tallam tmsri...@google.com wrote: Add new target builtins __builtin_cpu_is_intel_corei7

Re: [google][4.6]Compiler Directed Multiversioning with new -mvarch option (issue 5490054)

2011-12-17 Thread Sriraman Tallam
Committed to google 4_6 branch. Thanks, -Sri. On Sat, Dec 17, 2011 at 12:25 AM, Xinliang David Li davi...@google.com wrote: ok for google branches. David On Fri, Dec 16, 2011 at 2:05 PM,  tmsri...@google.com wrote: I have uploaded a new patch set with all the mentioned changes made. If a

[patch i386][google][4.6]Add new target builtins to check for corei7 and amdfam10 (issue5495075)

2011-12-16 Thread Sriraman Tallam
Add new target builtins __builtin_cpu_is_intel_corei7 and __builtin_cpu_is_amdfam10. * config/i386/i386-cpuinfo.c (__processor_model): Add new members __cpu_is_intel_corei7 and __cpu_is_amdfam10. (get_amd_cpu): Set __cpu_is_amdfam10. (get_intel_cpu): Set

Re: [4.6][google]Compiler Directed Function Multiversioning (issue 5477048)

2011-12-15 Thread Sriraman Tallam
I cc'ed gcc-patches by mistake. This is a patch still in progress. Please ignore. -Sri. On Thu, Dec 15, 2011 at 1:58 PM, tmsri...@google.com wrote: Reviewers: davidxl, Message: I have uploaded a new patch. I did not split the patch into two because I felt the ifunc dispatch patch does

[google][4.6]Compiler Directed Multiversioning with new -mvarch option (issue5490054)

2011-12-15 Thread Sriraman Tallam
Hi, I am working on user-directed and compiler-directed function multiversioning which has been discussed in these threads: 1) http://gcc.gnu.org/ml/gcc-patches/2011-04/msg02344.html 2) http://gcc.gnu.org/ml/gcc/2011-08/msg00298.html The gist of the discussions for user-directed

Re: [PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue 5488054)

2011-12-14 Thread Sriraman Tallam
On Wed, Dec 14, 2011 at 9:47 AM, Xinliang David Li davi...@google.com wrote: The patch is ok for google branches for now. Committed to google/main. Thanks, -Sri. David On Tue, Dec 13, 2011 at 4:33 PM,  tmsri...@google.com wrote: On 2011/12/14 00:04:10, davidxl wrote:

Re: [PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue5488054)

2011-12-13 Thread Sriraman Tallam
On Tue, Dec 13, 2011 at 9:58 AM, Richard Henderson r...@redhat.com wrote: On 12/12/2011 06:05 PM, Sriraman Tallam wrote: On core2, unaligned vector load/store using movdqu is a very slow operation. Experiments show it is six times slower than movdqa (aligned) and this is irrespective

Re: [PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue5488054)

2011-12-13 Thread Sriraman Tallam
On Mon, Dec 12, 2011 at 11:49 PM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Dec 12, 2011 at 06:05:57PM -0800, Sriraman Tallam wrote:       Do not vectorize loops on Core2 that need to use unaligned       vector load/stores.       * tree-vect-stmts.c (is_slow_vect_unaligned_load_store): New

Re: [PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue5488054)

2011-12-13 Thread Sriraman Tallam
On Mon, Dec 12, 2011 at 9:54 PM, Ira Rosen i...@il.ibm.com wrote: gcc-patches-ow...@gcc.gnu.org wrote on 13/12/2011 04:05:57 AM: On core2, unaligned vector load/store using movdqu is a very slow operation. Experiments show it is six times slower than movdqa (aligned) and this is

Re: [PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue5488054)

2011-12-13 Thread Sriraman Tallam
On Tue, Dec 13, 2011 at 10:56 AM, Richard Henderson r...@redhat.com wrote: On 12/13/2011 10:26 AM, Sriraman Tallam wrote: Cool, this works for stores!  It generates the movlps + movhps. I have to also make a similar change to another call to gen_sse2_movdqu for loads. Would it be ok to not do

Re: [PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue5488054)

2011-12-13 Thread Sriraman Tallam
On Tue, Dec 13, 2011 at 10:56 AM, Richard Henderson r...@redhat.com wrote: On 12/13/2011 10:26 AM, Sriraman Tallam wrote: Cool, this works for stores!  It generates the movlps + movhps. I have to also make a similar change to another call to gen_sse2_movdqu for loads. Would it be ok to not do

Re: [PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue5488054)

2011-12-13 Thread Sriraman Tallam
(info) propagation. In 4.4, gcc does alignment (output array) based versioning -- Sri's patch has the effect of doing the samething but only for selected targets. thanks, David On Tue, Dec 13, 2011 at 10:56 AM, Richard Henderson r...@redhat.com wrote: On 12/13/2011 10:26 AM, Sriraman

[PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue5488054)

2011-12-12 Thread Sriraman Tallam
On core2, unaligned vector load/store using movdqu is a very slow operation. Experiments show it is six times slower than movdqa (aligned) and this is irrespective of whether the resulting data happens to be aligned or not. For Corei7, there is no performance difference between the two and on

[google]Make test callgraph-profiles.C run only when section attribute e is supported (issue5167045)

2011-09-30 Thread Sriraman Tallam
Disable running of callgraph-profiles.C is section attribute e is not supported. Index: lib/target-supports-dg.exp === --- lib/target-supports-dg.exp (revision 179400) +++ lib/target-supports-dg.exp (working copy) @@ -100,6

Re: [google]Make test callgraph-profiles.C run only when section attribute e is supported (issue5167045)

2011-09-30 Thread Sriraman Tallam
, 2011 at 1:44 PM, Sriraman Tallam tmsri...@google.com wrote: Disable running of callgraph-profiles.C is section attribute e is not supported. Index: lib/target-supports-dg.exp === --- lib/target-supports-dg.exp  (revision 179400

Re: [google]Make test callgraph-profiles.C run only when section attribute e is supported (issue5167045)

2011-09-30 Thread Sriraman Tallam
On Fri, Sep 30, 2011 at 1:48 PM, Diego Novillo dnovi...@google.com wrote: On 11-09-30 16:44 , Sriraman Tallam wrote: Disable running of callgraph-profiles.C is section attribute e is not supported. OK with a ChangeLog entry. Was the original patch sent to trunk?  I don't recall

Re: [google]Make test callgraph-profiles.C run only when section attribute e is supported (issue5167045)

2011-09-30 Thread Sriraman Tallam
On Fri, Sep 30, 2011 at 1:51 PM, Diego Novillo dnovi...@google.com wrote: On 11-09-30 16:50 , Sriraman Tallam wrote: ChangeLog entry:        * testsuite/g++.dg/tree-prof/callgraph-profiles/C:      Add dg-require-section-exclude.     * testsuite/lib/target-supports.exp

[google] Specify target as ix86 or x86_64 for test builtin_target.c (issue5174041)

2011-09-30 Thread Sriraman Tallam
* testsuite/gcc.dg/builtin_target.c: Specify target as ix86 or x86_64. Index: gcc.dg/builtin_target.c === --- gcc.dg/builtin_target.c (revision 179400) +++ gcc.dg/builtin_target.c (working copy) @@ -1,6 +1,6

Re: [google] Specify target as ix86 or x86_64 for test builtin_target.c (issue5174041)

2011-09-30 Thread Sriraman Tallam
This is now submitted. Thanks, -Sri. On Fri, Sep 30, 2011 at 2:09 PM, Sriraman Tallam tmsri...@google.com wrote:        * testsuite/gcc.dg/builtin_target.c: Specify target as ix86 or        x86_64. Index: gcc.dg/builtin_target.c

Re: [google] Specify target as ix86 or x86_64 for test builtin_target.c (issue5174041)

2011-09-30 Thread Sriraman Tallam
Moving test gcc.target/i386 * testsuite/gcc.dg/builtin_target.c: Remove. * testsuite/gcc.target/i386/builtin_target.c: New file. On Fri, Sep 30, 2011 at 2:33 PM, Richard Henderson r...@redhat.com wrote: On 09/30/2011 02:09 PM, Sriraman Tallam wrote:       * testsuite/gcc.dg

Re: [PATCH] Add SECTION_EXCLUDE to list of section flags

2011-09-27 Thread Sriraman Tallam
Submitted to trunk after approval from Richard : http://gcc.gnu.org/ml/gcc-patches/2011-09/msg01747.html Thanks, -Sri. On Mon, Sep 26, 2011 at 2:09 PM, Sriraman Tallam tmsri...@google.com wrote: Hi,   Here is a simple patch to add SECTION_EXCLUDE to the list of section flag macros. Is this ok

Re: [google] Add SECTION_EXCLUDE flag and exclude .gnu.callgraph sections (issue5126041)

2011-09-27 Thread Sriraman Tallam
Submitted the SECTION_EXCLUDE part to trunk. Thanks, -Sri. On Tue, Sep 27, 2011 at 10:51 AM, Kai Tietz ktiet...@googlemail.com wrote: 2011/9/27 Richard Henderson r...@redhat.com: On 09/23/2011 04:45 PM, Sriraman Tallam wrote: I also want the SECTION_EXCLUDE part alone to be considered

Re: [google] Add SECTION_EXCLUDE flag and exclude .gnu.callgraph sections (issue5126041)

2011-09-27 Thread Sriraman Tallam
I committed the patch to google/gcc-4_6 branch. Thanks, -Sri. * output.h (SECTION_EXCLUDE): New flag for exclude sections. * varasm.c (default_elf_asm_named_section): Add e to section flags marked as SECTION_EXCLUDE. * final.c (rest_of_handle_final): Exclude

Re: [google] Linker plugin to do function reordering using callgraph edge profiles (issue5124041)

2011-09-27 Thread Sriraman Tallam
-if  as well. -Easwaran On Mon, Sep 26, 2011 at 5:22 PM, Sriraman Tallam tmsri...@google.com wrote: *Resending as plain text* I am attaching a patch of the updated files. This patch was meant for the google gcc-4_6 branch. Sorry for not mentioning this upfront in my original mail. Thanks

[PATCH] Add SECTION_EXCLUDE to list of section flags

2011-09-26 Thread Sriraman Tallam
Hi, Here is a simple patch to add SECTION_EXCLUDE to the list of section flag macros. Is this ok for trunk? Thanks, -Sri. * output.h (SECTION_EXCLUDE): New macro. * varasm.c (default_elf_asm_named_section): Check for SECTION_EXCLUDE. Index: gcc/varasm.c

Re: [google] Linker plugin to do function reordering using callgraph edge profiles (issue5124041)

2011-09-26 Thread Sriraman Tallam
*Resending as plain text* I am attaching a patch of the updated files. This patch was meant for the google gcc-4_6 branch. Sorry for not mentioning this upfront in my original mail. Thanks, -Sri. On Mon, Sep 26, 2011 at 9:53 AM, Sriraman Tallam tmsri...@google.com wrote: On Mon, Sep 26

[google] Linker plugin to do function reordering using callgraph edge profiles (issue5124041)

2011-09-23 Thread Sriraman Tallam
) @@ -0,0 +1,257 @@ +/* Callgraph implementation. + Copyright (C) 2011 Free Software Foundation, Inc. + Contributed by Sriraman Tallam (tmsri...@google.com). + +This program is free software; you can redistribute it and/or modify +it under the terms of the GNU General Public License as published

[google] Add SECTION_EXCLUDE flag and exclude .gnu.callgraph sections (issue5126041)

2011-09-23 Thread Sriraman Tallam
This patch adds a new flag to mark sections as exclude sections. Such sections will be discarded by the linker. Also, mark .gnu.callgraph sections which store the callgraph edge profile info as exclude sections. I also want the SECTION_EXCLUDE part alone to be considered for trunk. Index:

Preserve callee cgraph edges when callgraph profiles sections are requested. (issue5101042)

2011-09-22 Thread Sriraman Tallam
This patch preserves cgraph callee edges till pass_final if callgraph edge profiles sections are requested. It also renames callgraph edge profile sections to be .gnu.callgraph instead of .note.callgraph Index: cgraphbuild.c ===

Re: Preserve callee cgraph edges when callgraph profiles sections are requested. (issue5101042)

2011-09-22 Thread Sriraman Tallam
(gate_all_optimizations): Set cgraph_callee_edges_final_cleanup to true. (cgraph_callee_edges_final_cleanup): New global variable. * g++.dg/tree-prof/callgraph-profiles.C: New test. On Thu, Sep 22, 2011 at 4:01 PM, Sriraman Tallam tmsri...@google.com wrote: This patch preserves cgraph

[google][main]Fix broken test cases in google/main branch (issue4961065)

2011-09-06 Thread Sriraman Tallam
This patches fixes bugs that caused the multi-version tests to fail. * mversn-dispatch.c (specialize_call): Rebuild cgraph edges after specialization. (clone_and_dispatch_function): Rebuild cgraph edges to compute inline parameters.

Re: [4.7][google]Support for getting CPU type and feature information at run-time. (issue4893046)

2011-09-01 Thread Sriraman Tallam
, -Sri. thanks, David On Thu, Aug 25, 2011 at 5:37 PM, Sriraman Tallam tmsri...@google.com wrote: Hi,  Thanks for all the comments. I am attaching a new patch incorporating all of the changes mentioned, mainly : 1) Make __cpu_indicator_init a constructor in libgcc and guard to call

Re: [4.7][google]Support for getting CPU type and feature information at run-time. (issue4893046)

2011-08-26 Thread Sriraman Tallam
On Thu, Aug 25, 2011 at 6:02 PM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, Aug 25, 2011 at 5:37 PM, Sriraman Tallam tmsri...@google.com wrote: Hi,  Thanks for all the comments. I am attaching a new patch incorporating all of the changes mentioned, mainly : 1) Make __cpu_indicator_init

Re: [4.7][google]Support for getting CPU type and feature information at run-time. (issue4893046)

2011-08-26 Thread Sriraman Tallam
On Fri, Aug 26, 2011 at 10:10 AM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, Aug 26, 2011 at 10:06 AM, Sriraman Tallam tmsri...@google.com wrote: On Thu, Aug 25, 2011 at 6:02 PM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, Aug 25, 2011 at 5:37 PM, Sriraman Tallam tmsri...@google.com wrote: Hi

Re: [4.7][google]Support for getting CPU type and feature information at run-time. (issue4893046)

2011-08-26 Thread Sriraman Tallam
On Fri, Aug 26, 2011 at 10:24 AM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, Aug 26, 2011 at 10:17 AM, Sriraman Tallam tmsri...@google.com wrote: On Fri, Aug 26, 2011 at 10:10 AM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, Aug 26, 2011 at 10:06 AM, Sriraman Tallam tmsri...@google.com wrote

Re: [4.7][google]Support for getting CPU type and feature information at run-time. (issue4893046)

2011-08-26 Thread Sriraman Tallam
On Fri, Aug 26, 2011 at 10:45 AM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, Aug 26, 2011 at 10:37 AM, Sriraman Tallam tmsri...@google.com wrote: On Fri, Aug 26, 2011 at 10:24 AM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, Aug 26, 2011 at 10:17 AM, Sriraman Tallam tmsri...@google.com wrote

Re: [4.7][google]Support for getting CPU type and feature information at run-time. (issue4893046)

2011-08-22 Thread Sriraman Tallam
On Mon, Aug 22, 2011 at 9:02 AM, H.J. Lu hjl.to...@gmail.com wrote: On Mon, Aug 22, 2011 at 8:56 AM, Michael Matz m...@suse.de wrote: Hi, On Mon, 22 Aug 2011, H.J. Lu wrote: Oh, I thought it was data initialized by the constructor ... Sriramans patch right now has a function

Re: [4.7][google]Support for getting CPU type and feature information at run-time. (issue4893046)

2011-08-22 Thread Sriraman Tallam
On Mon, Aug 22, 2011 at 11:58 AM, H.J. Lu hjl.to...@gmail.com wrote: On Mon, Aug 22, 2011 at 11:50 AM, Sriraman Tallam tmsri...@google.com wrote: On Mon, Aug 22, 2011 at 9:02 AM, H.J. Lu hjl.to...@gmail.com wrote: On Mon, Aug 22, 2011 at 8:56 AM, Michael Matz m...@suse.de wrote: Hi, On Mon

Re: [4.7][google]Support for getting CPU type and feature information at run-time. (issue4893046)

2011-08-18 Thread Sriraman Tallam
On Thu, Aug 18, 2011 at 1:03 AM, Richard Guenther richard.guent...@gmail.com wrote: On Wed, Aug 17, 2011 at 7:54 PM, Sriraman Tallam tmsri...@google.com wrote: On Wed, Aug 17, 2011 at 12:37 AM, Richard Guenther richard.guent...@gmail.com wrote: On Tue, Aug 16, 2011 at 10:50 PM, Sriraman Tallam

Re: [4.7][google]Support for getting CPU type and feature information at run-time. (issue4893046)

2011-08-18 Thread Sriraman Tallam
On Thu, Aug 18, 2011 at 2:15 PM, Richard Henderson r...@redhat.com wrote: On 08/18/2011 10:25 AM, Sriraman Tallam wrote: Ok, so two things. I create the constructor as a comdat. So, it is created by gcc in every module but at link time only one copy will be kept. So, it is going to be called

Re: [4.7][google]Support for getting CPU type and feature information at run-time. (issue4893046)

2011-08-17 Thread Sriraman Tallam
On Wed, Aug 17, 2011 at 12:37 AM, Richard Guenther richard.guent...@gmail.com wrote: On Tue, Aug 16, 2011 at 10:50 PM, Sriraman Tallam tmsri...@google.com wrote: Support for getting CPU type and feature information at run-time. The following patch provides support for finding the platform type

Re: [4.7][google]Support for getting CPU type and feature information at run-time. (issue4893046)

2011-08-17 Thread Sriraman Tallam
On Wed, Aug 17, 2011 at 4:59 PM, Hans-Peter Nilsson h...@bitrange.com wrote: On Tue, 16 Aug 2011, Sriraman Tallam wrote: (I don't see anyone else making this comment, so maybe I missed something obvious, but I don't think so...) Support for getting CPU type and feature information at run

[4.7][google]Support for getting CPU type and feature information at run-time. (issue4893046)

2011-08-16 Thread Sriraman Tallam
-cpuinfo.c (revision 0) +++ libgcc/config/i386/i386-cpuinfo.c (revision 0) @@ -0,0 +1,275 @@ +/* Copyright (C) 2011 Free Software Foundation, Inc. + * Contributed by Sriraman Tallam tmsri...@google.com. + * + * This file is free software; you can redistribute it and/or modify it + * under

Re: [4.7][google]Support for getting CPU type and feature information at run-time. (issue4893046)

2011-08-16 Thread Sriraman Tallam
On Tue, Aug 16, 2011 at 2:06 PM, H.J. Lu hjl.to...@gmail.com wrote: On Tue, Aug 16, 2011 at 1:50 PM, Sriraman Tallam tmsri...@google.com wrote: Support for getting CPU type and feature information at run-time. The following patch provides support for finding the platform type at run-time

Re: [4.7][google]Support for getting CPU type and feature information at run-time. (issue4893046)

2011-08-16 Thread Sriraman Tallam
On Tue, Aug 16, 2011 at 3:35 PM, Joseph S. Myers jos...@codesourcery.com wrote: On Tue, 16 Aug 2011, Sriraman Tallam wrote: Index: libgcc/config/i386/t-cpuinfo === --- libgcc/config/i386/t-cpuinfo      (revision 0) +++ libgcc

Re: [google] New linker plugin to do function reordering in the final binary using callgraph profiles (issue4802070)

2011-08-02 Thread Sriraman Tallam
Hi, This patch adds a new linker plugin to re-order functions. The plugin constructs an annotated callgraph with edge profile information and then repeatedly groups sections that are connected by hot edges and passes the new function layout to the linker. The grouping is done similar to the

[patch,committed] Sync include/plugin-api.h with src/

2011-07-13 Thread Sriraman Tallam
Hi all, This patch was approved by Ian : http://sourceware.org/ml/binutils/2011-07/msg00136.html. I have synced plugin-api.h to gcc with the following patch in rev.176260 * plugin-api.h (ld_plugin_section): New struct. (ld_plugin_get_section_count): New typedef.

Re: [google] Patch to add compiler flag to dump callgraph edge profiles in special .note sections (issue4591045)

2011-06-08 Thread Sriraman Tallam
+davidxl On Tue, Jun 7, 2011 at 7:05 PM, Sriraman Tallam tmsri...@google.com wrote: Patch Description: = I am working on a project to do global function layout in the linker where the linker reads the callgraph edge profile information, generated by FDO, and uses

Re: [google] Patch to add compiler flag to dump callgraph edge profiles in special .note sections (issue4591045)

2011-06-08 Thread Sriraman Tallam
On Wed, Jun 8, 2011 at 9:16 AM, Xinliang David Li davi...@google.com wrote: ok for google/main. Thanks, the patch is now committed. David On Wed, Jun 8, 2011 at 9:13 AM, Sriraman Tallam tmsri...@google.com wrote: +davidxl On Tue, Jun 7, 2011 at 7:05 PM, Sriraman Tallam tmsri

[google] Patch to add compiler flag to dump callgraph edge profiles in special .note sections (issue4591045)

2011-06-07 Thread Sriraman Tallam
with data layout. Google ref 41940 2011-06-07 Sriraman Tallam tmsri...@google.com * doc/invoke.texi: document option -fcallgraph-profiles-sections. * final.c (dump_cgraph_profiles): New function. (rest_of_handle_final): Create new section '.note.callgraph.text

Re: [google] Patch to support calling multi-versioned functions via new GCC builtin. (issue4440078)

2011-05-04 Thread Sriraman Tallam
I submitted the patch. Thanks, -Sri. On Wed, May 4, 2011 at 3:13 PM, Diego Novillo dnovi...@google.com wrote: On Wed, May 4, 2011 at 15:35, Sriraman Tallam tmsri...@google.com wrote:        * tree-pass.h (pass_tree_convert_builtin_dispatch): New pass.        (pass_ipa_multiversion_dispatch

Re: [google] Patch to support calling multi-versioned functions via new GCC builtin. (issue4440078)

2011-05-02 Thread Sriraman Tallam
. Google ref 45947, 45986 2011-04-28  Sriraman Tallam  tmsri...@google.com        * c-family/c-common.c   (revision 173122) (handle_version_selector_attribute): New function.        (c_common_attribute_table): New attribute version_selector.        * tree-pass.h   (revision 173122

Re: [google] Patch to support calling multi-versioned functions via new GCC builtin. (issue4440078)

2011-04-29 Thread Sriraman Tallam
Hi Richard, Thanks for the comments. Please find inline responses. On Fri, Apr 29, 2011 at 1:56 AM, Richard Guenther richard.guent...@gmail.com wrote: On Fri, Apr 29, 2011 at 4:52 AM, Sriraman Tallam tmsri...@google.com wrote: I want this patch to be considered for google/main now

<    1   2   3   4