[PATCH] D81631: Fix undefined behavior in Dwarf.

zuojian lin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 9 04:23:59 PDT 2020


linzj added a comment.

> - bool SameAsPrevCU = this == DD->getPrevCU(); +  const DwarfCompileUnit *PrevCU = DD->getPrevCU();

Err, once you invoke getPrevCU, then a load-to-undefined instruction will be emitted. So  valgrind will still report this problem.  valgrind doesn't like undefined value's load.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81631





More information about the llvm-commits mailing list