[all-commits] [llvm/llvm-project] 762962: [Modules] Don't judge if we're compiling a module ...

Chuanqi Xu via All-commits all-commits at lists.llvm.org
Sun Sep 18 01:46:58 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 762962174e3adab1a0d744fe2636d4d9fb373c53
      https://github.com/llvm/llvm-project/commit/762962174e3adab1a0d744fe2636d4d9fb373c53
  Author: Chuanqi Xu <yedeng.yd at linux.alibaba.com>
  Date:   2022-09-18 (Sun, 18 Sep 2022)

  Changed paths:
    M clang/include/clang/Basic/LangOptions.h
    M clang/lib/Parse/ParseAST.cpp
    A clang/test/SemaCXX/PR57778.cpp

  Log Message:
  -----------
  [Modules] Don't judge if we're compiling a module unit by LangOpts::CurrentModule.empty()

Closing https://github.com/llvm/llvm-project/issues/57778.

Previously it judge if we're compiling a module unit by
LangOpts::CurrentModule.empty(). But it is not true since we can specify
the module name by `-fmodule-name` option for arbitrary module unit.
Then this patch adjuest the judgement properly.




More information about the All-commits mailing list