[PATCH] D74369: [AddressSanitizer] Ensure only AllocaInst is passed to dbg.declare

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 10 17:18:22 PST 2020


vsk planned changes to this revision.
vsk added a comment.

Apologies, it looks like the various ISel implementations do have some way to handle dbg.declares which point to arguments. In light of that it doesn't seem desirable to ban non-alloca dbg.declares outright. There might be a narrower condition that's worth enforcing, but that'll take more thought and analysis.

I'll work on incorporating feedback from @eugenis.


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

https://reviews.llvm.org/D74369





More information about the llvm-commits mailing list