[PATCH] D129832: [sanitizer] Add "mainsrc" prefix to sanitizer special case list

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 15 09:37:28 PDT 2022


MaskRay added a comment.



In D129832#3654436 <https://reviews.llvm.org/D129832#3654436>, @vitalybuka wrote:

> The patch is LGTM, but please consider to keep an ignorelist limited.

Thanks!

In D129832#3655393 <https://reviews.llvm.org/D129832#3655393>, @kstoimenov wrote:

> I think the name 'mainsrc' is slightly misleading because of the association with the 'main' function. Maybe something like primarysrc would be better to avoid this confusion?

How   about `mainfile`? Clang resources to the file as the `main file`. I picked `src` suffix just to be similar to `src`...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129832



More information about the cfe-commits mailing list