This is an archive of the discontinued LLVM Phabricator instance.

Fix variables.test after D49018
ClosedPublic

Authored by aleksandr.urakov on Jul 18 2018, 4:59 AM.

Details

Summary

This one fixes variables.test after D49018. The test was broken because D49018 adds a location information to variables, but I hadn't noticed that, because I used 32-bit build to run tests, so the test looked to me already broken before that commit (the test relies on mangled names, but the mangling schemes are different for 32-bit and 64-bit).

Diff Detail

Repository
rL LLVM

Event Timeline

stella.stamenova accepted this revision.Jul 18 2018, 8:17 AM
This revision is now accepted and ready to land.Jul 18 2018, 8:17 AM

Can you commit this for me, please? I have no commit access.

This revision was automatically updated to reflect the committed changes.