This is an archive of the discontinued LLVM Phabricator instance.

[asm] Make EmitMSInlineAsmStr and EmitGCCInlineAsmStr more alike
ClosedPublic

Authored by thakis on Nov 15 2021, 10:58 AM.

Details

Summary

https://reviews.llvm.org/D71677 copied a bunch of code from
EmitGCCInlineAsmStr() to EmitMSInlineAsmStr() but made a few small
(likely unintentional) changes. This makes these pieces look the same.

No behavior change.

(Why are these functions two copies? No great reason as far as I can tell.
https://reviews.llvm.org/rG1778831a3d1d24ab6545635f63da4d9c5f8f0ac7 did the
split; we might want to undo them at some point. But PR23933 suggests
that a bigger change is planned for this file in the future, so keeping
this incremental for now.)

Diff Detail

Event Timeline

thakis created this revision.Nov 15 2021, 10:58 AM
thakis requested review of this revision.Nov 15 2021, 10:58 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 15 2021, 10:58 AM
hans accepted this revision.Nov 15 2021, 11:21 AM

lgtm

This revision is now accepted and ready to land.Nov 15 2021, 11:21 AM
This revision was landed with ongoing or failed builds.Nov 15 2021, 12:47 PM
This revision was automatically updated to reflect the committed changes.