[PATCH] D97960: [clang-tidy] bugprone-signal-handler improvements: display call chain

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 5 07:18:41 PST 2021


njames93 added a comment.

The issue is FileCheck expects its directives to be in order they appear in the file being checked. Notes are always emitted just after the warning they are attached to.
So there are a few ways to go:

- use CHECK-NOTES-DAG to disregard the ordering and place them next to the location of the note in the code, to give correct line information.
- place the CHECK-NOTES directly after the warning they are attached.

Going with the first option makes it much harder to reason about.
Going with the second option you'll either have to just put a generic number regex for the line number, use (potentially large)  relative `@LINE` offsets or absolute line numbers.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97960



More information about the cfe-commits mailing list