[PATCH] D114826: [memprof] Align each rawprofile section to 8b.

Snehasish Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 30 20:12:18 PST 2021


snehasish added a comment.

In D114826#3163128 <https://reviews.llvm.org/D114826#3163128>, @vitalybuka wrote:

> In D114826#3163083 <https://reviews.llvm.org/D114826#3163083>, @snehasish wrote:
>
>> PTAL, thanks!
>
> It does not look like a trivial fix.
> If you need to wait @davidxl review I recommend to revert original patch and squash the fix into that one to unbreak buildbots
>
> BTW there is also https://lab.llvm.org/buildbot/#/builders/85/builds/7051/steps/7/logs/stdio

The breakage above should have been fixed by https://reviews.llvm.org/rGa2ce97cc3f99c8af80b2325acf03a4c171ffb48f. 
I'll go ahead and pushed this since David lgtm'ed it and keep an eye on the bot. Thanks for taking a look and your helpful suggestions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114826



More information about the llvm-commits mailing list