[PATCH] D92590: [CodeView] Fix inline sites that are missing code offsets.

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 3 12:41:53 PST 2020


dblaikie added inline comments.


================
Comment at: llvm/test/DebugInfo/COFF/inline-site-syms.ll:8-13
+; int test(int x) {
+;   auto f = [](int x) {
+;     return x * 100;
+;   };
+;   return f(x);
+; }
----------------
I'd probably suggest a simpler (it's probably about the same in IR, but simpler in terms of language features, names, etc) test case for inlining, something like:
```
void f1();
__attribute__((always_inline)) void f2() {
  f1();
}
void f3() {
  f2();
}
```

If that's adequate.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D92590/new/

https://reviews.llvm.org/D92590



More information about the llvm-commits mailing list