[PATCH] D137400: [llvm-debuginfo-analyzer] Fix format string-type mismatch in LVScope

Carlos Alberto Enciso via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 4 04:09:42 PDT 2022


CarlosAlbertoEnciso added a comment.

In D137400#3907490 <https://reviews.llvm.org/D137400#3907490>, @mgorny wrote:

> That said, I've only checked format strings in `LVScope.cpp`. I suspect other files may suffer from similar problems. If you replace `llvm::format()` with plain `printf()`, 32-bit GCC warns about type mismatches.

`PRIx64` it is a very good finding. I have the same feeling about the same issue in other modules. But at least we know the cause. Thanks for your fix.


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

https://reviews.llvm.org/D137400



More information about the llvm-commits mailing list