[PATCH] D64843: hwasan: Initialize the pass only once.

Peter Collingbourne via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 17 15:36:45 PDT 2019


pcc added a comment.

The instrumentation for globals is coming in a separate patch.

It seems more important to achieve implementation simplicity for the sanitizer passes rather than data locality because they make fundamental changes to the IR that touch the whole module (unlike other passes that are meant to preserve semantics). If the new PM is extended to allow whole-module changes within a function pass, I think the balance may change in favour of making this a function pass.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D64843





More information about the cfe-commits mailing list