[PATCH] D84215: [DFSan] Handle fast16labels for all API functions.

Matt Morehouse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 21 10:43:41 PDT 2020


morehouse added a comment.

In D84215#2164676 <https://reviews.llvm.org/D84215#2164676>, @kcc wrote:

> I think this is an overkill. 
>  fast16labels mode should be even simpler: 
>  there are always 16 primary labels, they don't have any descriptions or properties controlled by dfsan.


Seems reasonable, but note it will require effort for existing API users to switch.  For example, all the existing protobuf labeling API, etc. will need to be adapted.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84215





More information about the llvm-commits mailing list