[LLVMbugs] [Bug 8787] Support/Regex is unaware of LLP64(Win64)

[bugzilla-daemon at llvm.org]

http://llvm.org/bugs/show_bug.cgi?id=8787

Reid Kleckner <rnk at google.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |rnk at google.com
         Resolution|---                         |FIXED

--- Comment #3 from Reid Kleckner <rnk at google.com> ---
(In reply to comment #2)
> I have committed above as r121942.
> I think it is still workaround.
> I added FIXME and I will leave this issue opened.
> 
> /* FIXME: 'states' is assumed as 'long' on small version. */
> 
> Consider, to 'states, giving 'uint64_t' on i686,
> and giving 'short' on any hosts.

I don't think we need to do anything here. This workaround has held up fine for
four years, and by the time it stops working we can switch to C++11 <regex>.
This is still marked as blocking a win64 self-host, which already works.

Feel free to reopen if you feel otherwise.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20141106/907fc33f/attachment.html>