[PATCH] D84891: WIP: [Verifier] Flag dbg.declares which specify different addresses for the same fragment

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 31 17:47:26 PDT 2020


aprantl added a comment.

Without a testcase it's not immediately obvious what this is catching. It's legal for two *different* fragments of the same variable to point to two separate allocas, I think that's the output of SROA. It's not legal for any one variable (fragment) to be described by a dbg.declare and any other instrinsic. A dbg.declare is supposed to be the only, authoritative location of a variable. So maybe the test could be even stricter.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84891



More information about the llvm-commits mailing list