This is an archive of the discontinued LLVM Phabricator instance.

[ms-inline-asm] Fix bug in single asm statement support
ClosedPublic

Authored by myatsina on Feb 22 2016, 12:22 AM.

Details

Summary

Fixing a crash caused by trying to merge a single-line asm statement with an asm block that follows it, e.g:
asm int 4
asm {
int 5
}

Now, only adjacent single-line asm statements that are not surrounded by braces will be merged into one asm call.

Diff Detail

Repository
rL LLVM

Event Timeline

myatsina updated this revision to Diff 48651.Feb 22 2016, 12:22 AM
myatsina retitled this revision from to [ms-inline-asm] Fix bug in single asm statement support.
myatsina updated this object.
myatsina added reviewers: rnk, ehsan.
myatsina set the repository for this revision to rL LLVM.
myatsina added a subscriber: llvm-commits.
ehsan accepted this revision.Feb 22 2016, 12:39 PM
ehsan edited edge metadata.

Thanks, LGTM!

This revision is now accepted and ready to land.Feb 22 2016, 12:39 PM
This revision was automatically updated to reflect the committed changes.