[all-commits] [llvm/llvm-project] 7a0594: [pseudo] Remove the explicit Accept actions.

Haojian Wu via All-commits all-commits at lists.llvm.org
Thu Jun 9 02:20:46 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7a05942dd0c548cf4a0cf52d8d3f97b0a11a6509
      https://github.com/llvm/llvm-project/commit/7a05942dd0c548cf4a0cf52d8d3f97b0a11a6509
  Author: Haojian Wu <hokein.wu at gmail.com>
  Date:   2022-06-09 (Thu, 09 Jun 2022)

  Changed paths:
    M clang-tools-extra/pseudo/include/clang-pseudo/LRTable.h
    M clang-tools-extra/pseudo/lib/GLR.cpp
    M clang-tools-extra/pseudo/lib/grammar/LRTable.cpp
    M clang-tools-extra/pseudo/lib/grammar/LRTableBuild.cpp
    M clang-tools-extra/pseudo/test/lr-build-basic.test
    M clang-tools-extra/pseudo/test/lr-build-conflicts.test
    M clang-tools-extra/pseudo/unittests/GLRTest.cpp
    M clang-tools-extra/pseudo/unittests/LRTableTest.cpp

  Log Message:
  -----------
  [pseudo] Remove the explicit Accept actions.

As pointed out in the previous review section, having a dedicated accept
action doesn't seem to be necessary. This patch implements the the same behavior
without accept acction, which will save some code complexity.

Reviewed By: sammccall

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




More information about the All-commits mailing list