[all-commits] [llvm/llvm-project] 3262ae: [clang][Interp][NFC] Add missing return statement

Timm Bäder via All-commits all-commits at lists.llvm.org
Sun Jul 9 07:16:58 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3262aea06d36b1fbf3eb16d47afec742c18fd5e8
      https://github.com/llvm/llvm-project/commit/3262aea06d36b1fbf3eb16d47afec742c18fd5e8
  Author: Timm Bäder <tbaeder at redhat.com>
  Date:   2023-07-09 (Sun, 09 Jul 2023)

  Changed paths:
    M clang/lib/AST/Interp/Pointer.cpp

  Log Message:
  -----------
  [clang][Interp][NFC] Add missing return statement

I removed the else here, but forgot to add a return before.
Interestingly, none of the existing test cases triggered a problem. A
future commit will add a test case.


  Commit: 88d65e13464ab200f36082bf4f2613835e677144
      https://github.com/llvm/llvm-project/commit/88d65e13464ab200f36082bf4f2613835e677144
  Author: Timm Bäder <tbaeder at redhat.com>
  Date:   2023-07-09 (Sun, 09 Jul 2023)

  Changed paths:
    M clang/lib/AST/Interp/InterpBlock.h

  Log Message:
  -----------
  [clang][Interp][NFC] Fix a doc comment mixup


  Commit: fa1e9c3a5aacb1183fa3da410cb9dfd24b1a6d15
      https://github.com/llvm/llvm-project/commit/fa1e9c3a5aacb1183fa3da410cb9dfd24b1a6d15
  Author: Timm Bäder <tbaeder at redhat.com>
  Date:   2023-07-09 (Sun, 09 Jul 2023)

  Changed paths:
    M clang/lib/AST/Interp/InterpBlock.cpp
    M clang/lib/AST/Interp/InterpStack.cpp
    M clang/lib/AST/Interp/InterpState.cpp

  Log Message:
  -----------
  [clang][Interp][NFC] Add std:: qualifiers to all malloc/free calls


  Commit: b9e167994f01172c20716de28ea24f202b45b6a7
      https://github.com/llvm/llvm-project/commit/b9e167994f01172c20716de28ea24f202b45b6a7
  Author: Timm Bäder <tbaeder at redhat.com>
  Date:   2023-07-09 (Sun, 09 Jul 2023)

  Changed paths:
    M clang/lib/AST/Interp/Context.cpp

  Log Message:
  -----------
  [clang][Interp][NFC] Clear stack memory even on success

This makes it easier to use asan to find memory problems. clear() will
actually free the memory.


Compare: https://github.com/llvm/llvm-project/compare/048423702d4b...b9e167994f01


More information about the All-commits mailing list