[all-commits] [llvm/llvm-project] e7019f: [MLIR][IRDL] Instantiate "." in IRDL-defined names...

Ivan Ho via All-commits all-commits at lists.llvm.org
Sat Sep 26 03:56:47 PDT 2026


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e7019f85692bab245c02f4fa433b37105f2d9c0d
      https://github.com/llvm/llvm-project/commit/e7019f85692bab245c02f4fa433b37105f2d9c0d
  Author: Ivan Ho <ivan.ho at cl.cam.ac.uk>
  Date:   2026-09-26 (Sat, 26 Sep 2026)

  Changed paths:
    M mlir/lib/Dialect/IRDL/IR/IRDL.cpp
    M mlir/lib/Target/IRDLToCpp/IRDLToCpp.cpp
    M mlir/test/lib/Dialect/TestIRDLToCpp/test_irdl_to_cpp.irdl.mlir

  Log Message:
  -----------
  [MLIR][IRDL] Instantiate "." in IRDL-defined names as nested namespaces (#207035)

In PR #187911, it was revealed that the constraints on IRDL op names
placed by irdl-to-cpp were too restrictive.
This PR will address that by allowing "." in operation names to denote
nested namespaces, but not "$".

To discuss:
- ops like `wmma.f16.16x16x64.bf8_bf` cannot be registered as C++
identifiers cannot begin with a number. In discussions, we have opted to
prefix the generated namespaces with _.

Codex was used to generate the lit tests.

Close #187911.

Assisted-by: OpenAI Codex (GPT-5 and 6)



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list