[PATCH] D119777: [X86] Introduce x86-cmov-converter-force-all

Amir Ayupov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 17 10:33:58 PST 2022


Amir added a comment.

In D119777#3330058 <https://reviews.llvm.org/D119777#3330058>, @apostolakis wrote:

> Note Amir that the plan as outlined and discussed in the RFC that @MaskRay linked is to enable a new target-agnostic and profile-guided pass for cmov/branch decision-making and disable this profile-agnostic and x86-specific pass when profile information are available. Initially, the new pass will only be enabled for instr-FDO. But the next step is to enable it for Sample-FDO and as discussed with @modimo, this will take advantage of LBR data for computing misprediction rates. For the time being I am fine with enabling this option here but be aware that this x86 pass might be disabled in the near future.

Hi Sotiris,
Sounds good to me. The option added here is for experimentation ATM. Hoping to see the new pass with LBR data available soon!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119777



More information about the llvm-commits mailing list