This is an archive of the discontinued LLVM Phabricator instance.

[AArch64][v8.5A] Add speculation barrier to AArch64 instruction set
ClosedPublic

Authored by olista01 on Sep 25 2018, 7:16 AM.

Details

Summary

This is a new barrier which limits speculative execution of the
instructions following it.

Patch by Pablo Barrio!

Diff Detail

Repository
rL LLVM

Event Timeline

olista01 created this revision.Sep 25 2018, 7:16 AM
t.p.northover accepted this revision.Sep 26 2018, 7:55 AM

This looks fine.

This revision is now accepted and ready to land.Sep 26 2018, 7:55 AM
t.p.northover requested changes to this revision.Sep 26 2018, 7:57 AM

Actually, sorry. I think this should be marked HasSideEffects.

This revision now requires changes to proceed.Sep 26 2018, 7:57 AM
olista01 updated this revision to Diff 167146.Sep 26 2018, 8:34 AM

Add hasSideEffects.

t.p.northover accepted this revision.Sep 27 2018, 1:32 AM

Thanks. Looks fine now.

This revision is now accepted and ready to land.Sep 27 2018, 1:32 AM
This revision was automatically updated to reflect the committed changes.