[all-commits] [llvm/llvm-project] f584f0: [ConstExprPreter] Removed the flag forcing the use...

Nandor Licker via All-commits all-commits at lists.llvm.org
Wed Nov 27 12:13:41 PST 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: f584f04dab69ab15c8942753a145f0c6e7693bcc
      https://github.com/llvm/llvm-project/commit/f584f04dab69ab15c8942753a145f0c6e7693bcc
  Author: Nandor Licker <n at ndor.email>
  Date:   2019-11-27 (Wed, 27 Nov 2019)

  Changed paths:
    M clang/docs/ConstantInterpreter.rst
    M clang/include/clang/Basic/LangOptions.def
    M clang/include/clang/Driver/Options.td
    M clang/lib/AST/ExprConstant.cpp
    M clang/lib/AST/Interp/Context.cpp
    M clang/lib/AST/Interp/Context.h
    M clang/lib/Driver/ToolChains/Clang.cpp
    M clang/lib/Frontend/CompilerInvocation.cpp
    M clang/test/AST/Interp/cond.cpp

  Log Message:
  -----------
  [ConstExprPreter] Removed the flag forcing the use of the interpreter

Summary:
Removed the ```-fforce-experimental-new-constant-interpreter flag```, leaving
only the ```-fexperimental-new-constant-interpreter``` one. The interpreter
now always emits an error on an unsupported feature.

Allowing the interpreter to bail out would require a mapping from APValue to
interpreter memory, which will not be necessary in the final version. It is
more sensible to always emit an error if the interpreter fails.

Reviewers: jfb, Bigcheese, rsmith, dexonsmith

Subscribers: cfe-commits

Tags: #clang

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




More information about the All-commits mailing list