This is an archive of the discontinued LLVM Phabricator instance.

[ms] [X86] Use "P" modifier on all branch-target operands in inline X86 assembly.
ClosedPublic

Authored by epastor on Jan 8 2020, 2:50 PM.

Details

Summary

Extend D71677 to apply to all branch-target operands, rather than special-casing call instructions.

Also add a regression test for llvm.org/PR44272, since this finishes fixing it.

Diff Detail

Event Timeline

epastor created this revision.Jan 8 2020, 2:50 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJan 8 2020, 2:50 PM
epastor edited the summary of this revision. (Show Details)Jan 8 2020, 2:55 PM
epastor added reviewers: thakis, rnk.

Unit tests: pass. 61323 tests passed, 0 failed and 736 were skipped.

clang-tidy: fail. Please fix clang-tidy findings.

clang-format: fail. Please format your changes with clang-format by running git-clang-format HEAD^ or applying this patch.

Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml

thakis accepted this revision.Jan 8 2020, 6:30 PM
This revision is now accepted and ready to land.Jan 8 2020, 6:30 PM
This revision was automatically updated to reflect the committed changes.