[cfe-commits] Fix compilation error with static constexpr member functions

[Christopher Berner]

I've run into this bug (http://llvm.org/bugs/show_bug.cgi?id=13517), which seems to be related to the fixme in SemaType.cpp on line 2691.

I talked to Chandler Carruth, and he suggested that I look at Sema::MergeFunctionDecl. Moving all the logic there (like that fixme seems to imply) didn't seem to work, because then functions that have a single definition don't get const added to them.

Instead, I put logic there to remove the errant constness from static functions. I've attached a patch. Please let me know if you have any comments, as I've never contributed to the Clang codebase before.

Thanks!
Christopher Berner
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130103/2aaf13b9/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: clang.patch
Type: application/octet-stream
Size: 1811 bytes
Desc: clang.patch
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130103/2aaf13b9/attachment.obj>