Uploaded image for project: 'MariaDB Server'
  1. MariaDB Server
  2. MDEV-2678

LP:345524 - pbxt does not compile on 64 bit windows

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:

      Description

      PBXT does not compile on 64 bit windows due to use of inline assembly (64 bit Visual C++ compiler does not support inline assembly)
      Suggestion : use win32 atomics (InterlockedAdd etc) instead of assembly

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            paulmccullagh Paul McCullagh added a comment -

            Re: pbxt does not compile on 64 bit windows
            Hi Oleksandr,

            Ignore my last comment. Vlad has just pointed out the problem to me. The Windows compiler does not handle #ifdef in macros.

            I will push a fix shortly...

            Show
            paulmccullagh Paul McCullagh added a comment - Re: pbxt does not compile on 64 bit windows Hi Oleksandr, Ignore my last comment. Vlad has just pointed out the problem to me. The Windows compiler does not handle #ifdef in macros. I will push a fix shortly...
            Hide
            paulmccullagh Paul McCullagh added a comment -

            Re: pbxt does not compile on 64 bit windows
            Hi Oleksandr,

            The problems in ha_pbxt.cc should be fixed now.

            Please try...

            Show
            paulmccullagh Paul McCullagh added a comment - Re: pbxt does not compile on 64 bit windows Hi Oleksandr, The problems in ha_pbxt.cc should be fixed now. Please try...
            Hide
            sanja Oleksandr Byelkin added a comment -

            Re: pbxt does not compile on 64 bit windows
            Hi!

            Yes it fixed the problems. Thank you!

            Show
            sanja Oleksandr Byelkin added a comment - Re: pbxt does not compile on 64 bit windows Hi! Yes it fixed the problems. Thank you!
            Hide
            colin Colin Charles added a comment -

            Re: pbxt does not compile on 64 bit windows
            According to http://primebase.org/download/ChangeLog, this is fixed in 1.0.09f RC3 - 2009-11-30, and MariaDB-5.1.42 uses 1.0.09g so this bug is complete.

            Atomic operations are the only thing that you don't do on 64-bit windows now

            Show
            colin Colin Charles added a comment - Re: pbxt does not compile on 64 bit windows According to http://primebase.org/download/ChangeLog , this is fixed in 1.0.09f RC3 - 2009-11-30, and MariaDB-5.1.42 uses 1.0.09g so this bug is complete. Atomic operations are the only thing that you don't do on 64-bit windows now
            Hide
            ratzpo Rasmus Johansson added a comment -

            Launchpad bug id: 345524

            Show
            ratzpo Rasmus Johansson added a comment - Launchpad bug id: 345524

              People

              • Assignee:
                sanja Oleksandr Byelkin
                Reporter:
                wlad Vladislav Vaintroub
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: