[PATCH] D94333: [Inliner] Change inline remark format and update ReplayInlineAdvisor to use it

Di Mo via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 14 14:54:15 PST 2021


modimo added inline comments.


================
Comment at: llvm/lib/Analysis/InlineAdvisor.cpp:412
+
+  Remark << ";";
 }
----------------
wenlei wrote:
> nit: any special reason for adding this? doesn't seem consistent with other remarks we have.
If you grab the remark outputs via `-Rpass=inline` you'll get additional suffix information:
```
inline.cpp:8:12: remark: _Z3foov inlined into main with (cost=0, threshold=375) at callsite main:2:12; [-Rpass=inline]
    return foo();
```

The semicolon is to separate the remark from any additional output at the end so when replaying we can match the correct callsite. Something like this would be unneeded for yaml replay but for the current implementation it's necessary for correctness.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94333



More information about the cfe-commits mailing list