This is an archive of the discontinued LLVM Phabricator instance.

Added a workaround for a `-fdelayed-template-parsing` bug.
ClosedPublic

Authored by mpark on Jan 14 2017, 11:46 AM.

Details

Event Timeline

mpark updated this revision to Diff 84458.Jan 14 2017, 11:46 AM
mpark updated this revision to Diff 84459.
mpark retitled this revision from to Added a workaround for a `-fdelayed-template-parsing` bug..
mpark updated this object.
mpark added a reviewer: EricWF.

Fixed a variable name.

EricWF accepted this revision.Jan 14 2017, 12:02 PM
EricWF edited edge metadata.

LGTM!

Make sure to remove the // UNSUPPORTED: windows in the visit.pass.cpp test before committing.

This revision is now accepted and ready to land.Jan 14 2017, 12:02 PM
EricWF added inline comments.Jan 14 2017, 1:18 PM
include/variant
483

Oh this should also use _VSTD::addressof

mpark added inline comments.Jan 16 2017, 12:17 AM
include/variant
483

I did this just because of https://godbolt.org/g/4yYeXu, but I'll add it for now.

mpark updated this revision to Diff 84520.Jan 16 2017, 12:20 AM
mpark edited edge metadata.
  • Removed the // UNSUPPORTED: windows in visit.pass.cpp.
  • Used _VSTD::addressof.
mpark marked 2 inline comments as done.Jan 16 2017, 12:22 AM
This revision was automatically updated to reflect the committed changes.