This is an archive of the discontinued LLVM Phabricator instance.

[ASTImporter] Added ability to import AtomicType nodes
ClosedPublic

Authored by khazem on Nov 5 2016, 8:34 PM.

Details

Summary

This patch fixes an issue where attempting to import code containing atomic type declarations would result in the error "cannot import unsupported AST node Atomic".

Diff Detail

Repository
rL LLVM

Event Timeline

khazem updated this revision to Diff 76976.Nov 5 2016, 8:34 PM
khazem retitled this revision from to [ASTImporter] Added ability to import AtomicType nodes.
khazem updated this object.
khazem added reviewers: spyffe, ABataev.
khazem added subscribers: phosek, smklein.
spyffe accepted this revision.Nov 7 2016, 10:36 AM
spyffe edited edge metadata.

Looks good to me. Thanks, Kareem!

This revision is now accepted and ready to land.Nov 7 2016, 10:36 AM

Thank you Sean! I've asked Petr Hosek to commit this change.

Can he commit this? In case he is too busy, I can also commit this for you.

Thanks, Gábor! Please do commit this (it's currently Thanksgiving week in the US).

This revision was automatically updated to reflect the committed changes.