This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt] Change all asm directives to __asm__ in cpu_model.
ClosedPublic

Authored by asbirlea on Jul 17 2016, 1:36 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

asbirlea updated this revision to Diff 64243.Jul 17 2016, 1:36 AM
asbirlea retitled this revision from to [compiler-rt] Change all asm directives to __asm__ in cpu_model..
asbirlea updated this object.
vitalybuka accepted this revision.Jul 17 2016, 1:36 PM
vitalybuka edited edge metadata.

lgtm

This revision is now accepted and ready to land.Jul 17 2016, 1:36 PM
compnerd added inline comments.
lib/builtins/cpu_model.c
230 ↗(On Diff #64243)

This would be nice to leave as __asm as Microsoft documents it as __asm blocks. https://msdn.microsoft.com/en-us/library/45yd4tzz.aspx

asbirlea updated this revision to Diff 64266.Jul 17 2016, 4:40 PM
asbirlea edited edge metadata.

Leave __asm for MSVC.

compnerd accepted this revision.Jul 17 2016, 4:49 PM
compnerd added a reviewer: compnerd.
This revision was automatically updated to reflect the committed changes.