[all-commits] [llvm/llvm-project] 62ef1c: [flang][msvc] Remove default arguments for functio...

Michael Kruse via All-commits all-commits at lists.llvm.org
Wed Aug 12 14:46:22 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 62ef1cb2079123b86878e4bfed3c14db448f1373
      https://github.com/llvm/llvm-project/commit/62ef1cb2079123b86878e4bfed3c14db448f1373
  Author: Michael Kruse <llvm-project at meinersbur.de>
  Date:   2020-08-12 (Wed, 12 Aug 2020)

  Changed paths:
    M flang/include/flang/Decimal/decimal.h

  Log Message:
  -----------
  [flang][msvc] Remove default arguments for function specializations.

C++ does not allow function template specializations to have default arguments. IIUC callers will only use the default arguments of the primary template. gcc and clang accept the syntax anyway, but msvc emits an error.

See https://en.cppreference.com/w/cpp/language/template_specialization for more details.

This patch is part of the series to [[ http://lists.llvm.org/pipermail/flang-dev/2020-July/000448.html | make flang compilable with MS Visual Studio ]].

Reviewed By: DavidTruby

Differential Revision: https://reviews.llvm.org/D85657




More information about the All-commits mailing list