[llvm] [MemProf] Add option to match summary and definition strictly (PR #83665)

via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 6 23:31:53 PST 2024


lifengxiang1025 wrote:

> Thanks for working on this. Since I believe the test case will assert without the added options, I'm thinking it is probably best to simply always do this in a debug compiler. Can we force enable enable-import-metadata when cloning is enabled in the thin link (or pass down a flag to ComputeCrossModuleImport to enable it)? The checking that fails without this patch should probably also go under an `#ifndef NDEBUG` (looks like I missed adding that).

Sorry, I don't understand fully.
1. I have updated code to match definition and summary default. But I can't understand why we need to distinguish debug compiler and release compiler.
2. I don't understand where to add `#ifndef NDEBUG`.


https://github.com/llvm/llvm-project/pull/83665


More information about the llvm-commits mailing list