[clang] [llvm] [Inliner] Propagate more attributes to params when inlining (PR #91101)

via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 2 07:57:31 PDT 2024


goldsteinn wrote:

> It may be a good idea to skip the inference for constant arguments. I see a decent amount of things like `range(i64 -2147483576, 34359738361) 272` in the diffs.
> 
Will skip constants.

> Though I still don't think that we should be inferring range at all.

IMO throwing away information that may be irrecoverable should be avoid if its not necessary for the transform.

https://github.com/llvm/llvm-project/pull/91101


More information about the cfe-commits mailing list