[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