On 08/05/2015 10:02 AM, Jiong Wang wrote:
On 28/07/15 16:44, Martin Sebor wrote:
Attached is an updated patch with the changes above.
gcc/testsuite/ChangeLog
2015-07-28 Martin Sebor<mse...@redhat.com>
* g++.dg/Wframe-address-in-Wall.C: New test.
* g++.dg/Wframe-address.C: New test.
* g++.dg/Wno-frame-address.C: New test.
* gcc.dg/Wframe-address-in-Wall.c: New test.
* gcc.dg/Wframe-address.c: New test.
* gcc.dg/Wno-frame-address.c: New test.
noticed the new added "Wno-frame-address.c" fail on
arm-none-linux-gnueabihf native test.
from the comments in the testcase:
+/* Verify that -Wframe-address is not enabled by default by enabling
+ -Werror and verifying the test still compiles. */
seems you want to make sure -Wframe-address work correctly with -Werror,
while for arm,
return_address hook is defined to only support level 0, NULL_RTX
returned for all other levels,
so this caused Wno-frame-address.c failed in those tem != NULL check for
builtin_return_address.
Thanks. I'm traveling the next 10 days and most likely won't be
able to get to this until I get back on the 17th.
Martin
