[PATCH] D81631: Fix undefined behavior in Dwarf.

[David Blaikie via Phabricator via llvm-commits]

dblaikie added a comment.

In D81631#2094574 <https://reviews.llvm.org/D81631#2094574>, @linzj wrote:

> In D81631#2093513 <https://reviews.llvm.org/D81631#2093513>, @dblaikie wrote:
>
> > Is this UB codepath reached by existing test cases running in check-llvm? Or was it found on some other testing?
>
>
> No. I ran my compiler which is based on LLVM with valgrind. Then this warning was popping everywhere.


Please ensure/add a test case to the LLVM regression tests that demonstrates the codepath to ensure it has some coverage/regression testing.  (if this doesn't reproduce with Clang's IR/LLVM's regression tests, it's possible your compiler is producing some novel IR that we're not testing/haven't thought about)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81631