PING [PATCH][gcc][PR94230]provide an option to change the size limitation for -Wmisleading-indent

[Qing Zhao]

Hi,

Please take a look at the attached patch and let me know your comments.

Thanks.

Qing

gcc/ChangeLog:

2020-04-03  qing zhao  <qing.zhao@oracle.com>

	* common.opt: Add -flocation-ranges.
	* doc/invoke.texi: Document it.
	* toplev.c (process_options): set line_table->default_range_bits
	to 0 when flag_location_ranges is false. 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: PR94230.patch
Type: application/octet-stream
Size: 2183 bytes
Desc: not available
URL: <https://gcc.gnu.org/pipermail/gcc-patches/attachments/20200408/badc3c01/attachment.obj>
-------------- next part --------------


> Begin forwarded message:
> 
> From: Qing Zhao via Gcc-patches <gcc-patches@gcc.gnu.org>
> Subject: [PATCH][gcc][PR94230]provide an option to change the size limitation for -Wmisleading-indent 
> Date: April 3, 2020 at 2:55:53 PM CDT
> To: dmalcolm@redhat.com, jakub@redhat.com
> Cc: gcc-patches@gcc.gnu.org
> Reply-To: Qing Zhao <QING.ZHAO@ORACLE.COM>
> 
> Hi, David and Jakub,
> 
> Per the discussion we had for PR94230: provide an option to change the size limitation for -Wmisleading-indent
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94230 <https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94230>
> 
> I come up with the following simple patch per David?s suggestion:
> 
> Provide a new first class option -flocation-ranges to control enabling/disablng range tracking when recording source locations.
> The default value for this option is enabling the range tracking.
> 
> When specify -fno-location-ranges, disable the range tracking to save space for column tracking. 
> 
> I have tested this GCC to build our huge application by adding -fno-location-ranges, the whole build completed without any issue. and
> -Wmisleading-indent also emitted several helpful warning message during building.
> 
> I am running bootstrap right now.
> 
> Could you please take a look at the attached patch?
> 
> thanks.
> 
> Qing
> 
> 
> 
>