This is an archive of the discontinued LLVM Phabricator instance.

[X86] Mark builtins 'const' where possible
ClosedPublic

Authored by craig.topper on May 1 2018, 10:12 PM.

Details

Summary

I attempted to go through all the builtins and marked them const if they didn't touch memory or other hidden state.

I don't know how to test this either or if it really matters.

Diff Detail

Repository
rC Clang

Event Timeline

craig.topper created this revision.May 1 2018, 10:12 PM
efriedma accepted this revision.May 8 2018, 1:46 PM
efriedma added a subscriber: efriedma.

I'm pretty sure this has zero visible effect, but I guess it makes sense as documentation. LGTM.

This revision is now accepted and ready to land.May 8 2018, 1:46 PM
This revision was automatically updated to reflect the committed changes.