[PATCH] D75768: Add a warning for builtin_return_address/frame_address with > 0 argument

Jeremy Stenglein via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 6 13:15:50 PST 2020


jstenglein added a comment.

In D75768#1910303 <https://reviews.llvm.org/D75768#1910303>, @erichkeane wrote:

> Any reason to not put this in -Wall like GCC? I'm not terribly knowledgeable about the intended guidance for adding to Wmost, so if you can clarify this decision I'd be grateful.  Otherwise I think this patch looks fine.


Thanks for the comments.  It is already part of -Wall since I put it in -Wmost and -Wall includes -Wmost:
def All : DiagGroup<"all", [Most, Parentheses, Switch, SwitchBool, MisleadingIndentation]>;


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75768/new/

https://reviews.llvm.org/D75768





More information about the cfe-commits mailing list