[all-commits] [llvm/llvm-project] 92a7f6: [lldb][rpc] Fix bug in convert script for RPC (#14...

[S. VenkataKeerthy via All-commits]

  Branch: refs/heads/users/svkeerthy/revert-06-10-_mlininer_ir2vec_integrating_ir2vec_with_mlinliner
  Home:   https://github.com/llvm/llvm-project
  Commit: 92a7f6fbbe489c02bad77a41cf2d4154c3885796
      https://github.com/llvm/llvm-project/commit/92a7f6fbbe489c02bad77a41cf2d4154c3885796
  Author: Chelsea Cassanova <chelsea_cassanova at apple.com>
  Date:   2025-06-23 (Mon, 23 Jun 2025)

  Changed paths:
    M lldb/scripts/convert-lldb-header-to-rpc-header.py
    M lldb/test/Shell/RPC/Scripts/TestConvertScript/CheckLLDBEnumerations.test
    M lldb/test/Shell/RPC/Scripts/TestConvertScript/Inputs/lldb-enumerations.h

  Log Message:
  -----------
  [lldb][rpc] Fix bug in convert script for RPC (#145419)

In the script that's used by RPC to convert LLDB headers to LLDB RPC
headers, there's a bug with how it converts namespace usage. An
overeager regex pattern caused *all* text before any `lldb::` namespace
usage to get replaced with `lldb_rpc::` instead of just the namespace
itself. This commit changes that regex pattern to be less overeager and
modifies one of the shell tests for this script to actually check that
the namespace usage replacement is working correctly.

rdar://154126268


  Commit: d4299941959c28e08a9474aceeb5fce10e26f431
      https://github.com/llvm/llvm-project/commit/d4299941959c28e08a9474aceeb5fce10e26f431
  Author: S. VenkataKeerthy <31350914+svkeerthy at users.noreply.github.com>
  Date:   2025-06-24 (Tue, 24 Jun 2025)

  Changed paths:
    M lldb/scripts/convert-lldb-header-to-rpc-header.py
    M lldb/test/Shell/RPC/Scripts/TestConvertScript/CheckLLDBEnumerations.test
    M lldb/test/Shell/RPC/Scripts/TestConvertScript/Inputs/lldb-enumerations.h

  Log Message:
  -----------
  Merge branch 'main' into users/svkeerthy/revert-06-10-_mlininer_ir2vec_integrating_ir2vec_with_mlinliner


Compare: https://github.com/llvm/llvm-project/compare/1907bd398d3b...d4299941959c

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