[PATCH] D157615: [ExtendLifetimes][1/4] Add "disable-post-ra" function attribute to disable the post-regalloc scheduler

Mon Sep 4 01:50:43 PDT 2023

StephenTozer added a comment.

In D157615#4634448 <https://reviews.llvm.org/D157615#4634448>, @jmorse wrote:

> Do we need to register this attribute somewhere, in a .td file or something, to ensure it's discoverable by other people performing maintenance?

Yeah, I think there are a few missing spots in this patch for documentation and bitcode representation that I'll fill in now.

> Possibly a (bikeshed warning) more abstract name like "prioritize-debuggability" that's then interpreted by the post-ra scheduler would be better. No strong opinion from me.

Bikeshedding concerns accepted given the discussion on the current discourse RFC - an attribute like `prioritize-debuggability` or `opt-debug` makes sense, since that also helps solve the "passing optimization info through LTO" problem.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157615