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.
Paths
| Differential D46349
[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
Event Timelinecraig.topper added a parent revision: D46328: [X86] Mark all x86 specific builtins as nothrow..May 2 2018, 11:37 AM Comment Actions 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 Closed by commit rC331814: [X86] Mark builtins 'const' where possible (authored by ctopper). · Explain WhyMay 8 2018, 3:07 PM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 145796 include/clang/Basic/BuiltinsX86.def
include/clang/Basic/BuiltinsX86_64.def
|