[PATCH] D143311: [MLGO] Add BB Profile Dump Pass for Regalloc Case

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 11 08:07:54 PST 2023


mtrofin added a comment.

In D143311#4120232 <https://reviews.llvm.org/D143311#4120232>, @thakis wrote:

> The enable of this test without tflite broke tests on Mac: http://45.33.8.238/macm1/54722/step_11.txt
>
> Please take a look and revert for now if it takes a while to fix.
>
> (Saying this here since the other change doesn't have a phab link in its commit message.)

@thakis can you confirm D143815 <https://reviews.llvm.org/D143815> - on my mac (x86), the test passed even before the proposed patch, and I'm not able to glean from your bot what (if anything) could be different between my setup and yours - I looked into http://45.33.8.238/macm1/54722/log.txt and http://45.33.8.238/macm1/54722/, looking for a `gn ` or `gn config`.

My setup is as vanilla as `cmake -GNinja ../llvm -DCMAKE_BUILD_TYPE=Release`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143311



More information about the llvm-commits mailing list