This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt] Fix broken unittest on win64
ClosedPublic

Authored by etienneb on Jul 20 2016, 8:00 PM.

Details

Summary

This is an other tentative to fix:
https://reviews.llvm.org/D22588

It's less clever, but should work.

Turn out there is not an easy way to write a portable print
for a pointer in lowercase without the prefix 0x.

Diff Detail

Event Timeline

etienneb updated this revision to Diff 64811.Jul 20 2016, 8:00 PM
etienneb retitled this revision from to [compiler-rt] Fix broken unittest on win64.
etienneb updated this object.
etienneb added a reviewer: rnk.
rnk accepted this revision.Jul 21 2016, 7:11 AM
rnk edited edge metadata.

OK, let's do it the less fancy way. =/ lgtm

test/asan/TestCases/debug_mapping.cc
24

This might produce -Wformat warnings, but I guess we don't care.

test/asan/TestCases/debug_stacks.cc
44

No need to cast, this is already a void*, right?

This revision is now accepted and ready to land.Jul 21 2016, 7:11 AM
etienneb updated this revision to Diff 64891.Jul 21 2016, 7:40 AM
etienneb marked 2 inline comments as done.
etienneb edited edge metadata.

fix rnk@ comments

etienneb closed this revision.Jul 21 2016, 7:43 AM