[clang] [llvm] [NFC] Use const members of `StringToOffsetTable` (PR #105824)

2024-08-25 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `sanitizer-x86_64-linux` running on `sanitizer-buildbot2` while building `clang,llvm` at step 2 "annotate". Full details are available at: https://lab.llvm.org/buildbot/#/builders/66/builds/3089 Here is the relevant piece of

[clang] [llvm] [NFC] Use const members of `StringToOffsetTable` (PR #105824)

2024-08-25 Thread Rahul Joshi via cfe-commits
https://github.com/jurahul closed https://github.com/llvm/llvm-project/pull/105824 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [NFC] Use const members of `StringToOffsetTable` (PR #105824)

2024-08-23 Thread Sergei Barannikov via cfe-commits
https://github.com/s-barannikov approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/105824 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [NFC] Use const members of `StringToOffsetTable` (PR #105824)

2024-08-23 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Rahul Joshi (jurahul) Changes - After the table is populated and emitted as a string, we do not expect any modifications to the table. So use const reference and const members to access the table from that point on. --- Full diff: https:

[clang] [llvm] [NFC] Use const members of `StringToOffsetTable` (PR #105824)

2024-08-23 Thread Rahul Joshi via cfe-commits
https://github.com/jurahul ready_for_review https://github.com/llvm/llvm-project/pull/105824 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [NFC] Use const members of `StringToOffsetTable` (PR #105824)

2024-08-23 Thread Rahul Joshi via cfe-commits
https://github.com/jurahul created https://github.com/llvm/llvm-project/pull/105824 - After the table is populated and emitted as a string, we do not expect any modifications to the table. So use const reference and const members to access the table from that point on. >From 154c0728820a8a6e1