Revert "[clang][Interp][NFC] Save source location of evaluating expression"

This reverts commit de04e6c.

This breaks the Misc/constexpr-source-ranges test:
https://lab.llvm.org/buildbot/#/builders/139/builds/64685

Author: tbaederr

clang/lib/AST/Interp/EvalEmitter.cpp

@@ -34,7 +34,6 @@ EvalEmitter::~EvalEmitter() {
 
 EvaluationResult EvalEmitter::interpretExpr(const Expr *E,
                                             bool ConvertResultToRValue) {
-  S.setEvalLocation(E->getExprLoc());
   this->ConvertResultToRValue = ConvertResultToRValue;
   EvalResult.setSource(E);
 

clang/lib/AST/Interp/InterpFrame.cpp

@@ -191,11 +191,8 @@ Frame *InterpFrame::getCaller() const {
 }
 
 SourceRange InterpFrame::getCallRange() const {
-  if (!Caller->Func) {
-    if (S.EvalLocation.isValid())
-      return S.EvalLocation;
+  if (!Caller->Func)
     return S.getRange(nullptr, {});
-  }
   return S.getRange(Caller->Func, RetPC - sizeof(uintptr_t));
 }
 

clang/lib/AST/Interp/InterpState.h

@@ -98,8 +98,6 @@ class InterpState final : public State, public SourceMapper {
 
   Context &getContext() const { return Ctx; }
 
-  void setEvalLocation(SourceLocation SL) { this->EvalLocation = SL; }
-
 private:
   /// AST Walker state.
   State &Parent;
@@ -117,8 +115,6 @@ class InterpState final : public State, public SourceMapper {
   Context &Ctx;
   /// The current frame.
   InterpFrame *Current = nullptr;
-  /// Source location of the evaluating expression
-  SourceLocation EvalLocation;
 };
 
 } // namespace interp