[PATCH] D144136: Add a "remark" to report on array accesses

Bill Wendling via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 16 12:24:53 PST 2023


void added a comment.

In D144136#4131825 <https://reviews.llvm.org/D144136#4131825>, @aaron.ballman wrote:

> I'd like to understand what the overhead is for this. How much overhead does this add when the remark is disabled? How much overhead does this add when the remark is enabled?
>
>> This will report a ton of information. It's basically only good for piping to a file and using Perl to gather any useful information.
>
> This is why I'm worried about overhead -- there can be *a lot* of array accesses in a complex TU and spitting data out to (potentially) stdout/stderr is *slow*.

I should have indicated that I'm not expecting this to go into mainline Clang. As you mentioned, this would be a lot of overhead and is really only useful in the way @kees mentioned. I created this "arc" patch to make it easier for him to import it. :-) Sorry for the confusion.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144136



More information about the cfe-commits mailing list