[PATCH] D100738: Cache attribute checks

[Duncan P. N. Exon Smith via Phabricator via llvm-commits]

dexonsmith added a comment.

I triggered a run in the compile-time tracker for this patch:
http://llvm-compile-time-tracker.com/compare.php?from=a7712091ea7a2b4a7b5c4764af7545a7574b651b&to=f01acba0cdc4625738be504449a71704595cd12f&stat=instructions

Looks like an improvement, but I'm not how link time is exposed in that interface... or maybe it isn't? @nikic, you mentioned there was a 1% regression in link time: can you confirm if this fixes it?

Note I also triggered a run where attributes themselves are also cached:
http://llvm-compile-time-tracker.com/compare.php?from=f01acba0cdc4625738be504449a71704595cd12f&to=68b7232d1d68abb75486de8588094742c3146ba9&stat=instructions
Seems more noisy.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100738